-
Notifications
You must be signed in to change notification settings - Fork 281
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
add rot13 #80
add rot13 #80
Conversation
Excellent job @pgrek001 for adding a new converter here! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the tests fail, it's due to my latest change. It won't break anything existing though. If you want to pre-emptively fix, merge main and change the return type to match, but I'm happy to fix it up. Sorry, should have postponed my latest pull until this went in!
I quickly fixed it (I think) |
Description
Tests
Documentation