The Wayback Machine - https://web.archive.org/web/20210605234359/https://github.com/TheAlgorithms/Java/pull/2215
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2212 #2215

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

2212 #2215

wants to merge 3 commits into from

Conversation

@anishsaraogi
Copy link

@anishsaraogi anishsaraogi commented May 22, 2021

Issue #2212 fixed and Password Validator added.

@esparev
Copy link

@esparev esparev commented May 23, 2021

Technically, it's not wrong, and the code wasn't broken, but it isn't working the way it's supposed to with the changes you made.

@esparev
Copy link

@esparev esparev commented May 23, 2021

I suggest you look again at the code that I have and I strongly recommend that you give the variables a more explicit and explanatory name.

@anishsaraogi
Copy link
Author

@anishsaraogi anishsaraogi commented May 24, 2021

It's probably the special characters because as I saw, your code includes more special characters than I put. I will add it asap.

Made the changes as required.
@esparev
Copy link

@esparev esparev commented May 25, 2021

Appreciate the attention, thank you @anishsaraogi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants