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

Remove invalid escape sequence from docstring #296

Conversation

jku
Copy link
Collaborator

@jku jku commented Oct 30, 2020

generate_public_and_private() docstring contains an invalid escape sequence which can lead to an annoying
DeprecationWarning: invalid escape sequence
Remove the whole string as it does not seem very useful.


The related issue is #290: this fixes half of it. Updating the vendored ed25519 will fix the other instance.

@coveralls
Copy link

coveralls commented Oct 30, 2020

Coverage Status

Coverage remained the same at 98.868% when pulling a3339cc on jku:fix-invalid-escape-sequence-in-docstring into ec9c5a4 on secure-systems-lab:master.

generate_public_and_private() docstring contains an invalid escape
sequence which can lead to an annoying
    DeprecationWarning: invalid escape sequence
Remove the whole string as it does not seem very useful.
@jku jku force-pushed the fix-invalid-escape-sequence-in-docstring branch from a3339cc to 3c0fd2e Compare October 30, 2020 09:31
@lukpueh
Copy link
Member

lukpueh commented Oct 30, 2020

🙇

@lukpueh lukpueh merged commit d4fef67 into secure-systems-lab:master Oct 30, 2020
@lukpueh lukpueh mentioned this pull request Nov 9, 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.

3 participants