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

Adopt interface changes (#288) in README snippets #300

Merged
merged 1 commit into from
Nov 9, 2020

Conversation

lukpueh
Copy link
Member

@lukpueh lukpueh commented Nov 9, 2020

Description of the changes being introduced by the pull request:

Adopt the following changes:

  • generate_and_write_*_keypair no longer auto-prompts for a
    password generate_and_write_*_keypair_with_prompt should be
    used to present a prompt.
  • import_*_privatekey_from_file has a new prompt message.

Please verify and check that the pull request fulfils the following requirements:

  • The code follows the Code Style Guidelines
  • Tests have been added for the bug fix or new feature
  • Docs have been added for the bug fix or new feature

Adopt the following changes:
- generate_and_write_*_keypair no longer auto-prompts for a
  password generate_and_write_*_keypair_with_prompt should be
  used to present a prompt.
- import_*_privatekey_from_file has a new prompt message.
@coveralls
Copy link

coveralls commented Nov 9, 2020

Coverage Status

Coverage remained the same at 98.785% when pulling 43deff7 on lukpueh:adopt-288-in-readme into 92da669 on secure-systems-lab:master.

@lukpueh lukpueh mentioned this pull request Nov 9, 2020
@lukpueh lukpueh requested a review from joshuagl November 9, 2020 14:04
Copy link
Collaborator

@joshuagl joshuagl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lukpueh lukpueh merged commit deb50ad into secure-systems-lab:master 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