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

adds Rot 13 Cypher project #82

Merged

Conversation

Capocaccia
Copy link
Contributor

No description provided.

@thinkswell
Copy link
Owner

Hey @Capocaccia , Your project is good.
But while encrypting I think you missed the last letter, cause on decrypting the encrypted text it is not showing the last letter of my string.
I would also like to suggest you, do consider the case of the letters.
Like: Hello -> Uryy
Uryy -> Hello
Your current code giving: Urry -> hell
Please look out to this.
Thank You! 😄

@Capocaccia
Copy link
Contributor Author

@thinkswell I have corrected these issues. Please take another look when you have a moment. Thank you.

@thinkswell
Copy link
Owner

Looks Excellent! 🎉

@thinkswell thinkswell merged commit 9ac9e41 into thinkswell:master Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants