-
Notifications
You must be signed in to change notification settings - Fork 87
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
Update README.md #51
Update README.md #51
Conversation
@@ -71,7 +71,7 @@ The `gokey` binary should appear in your `$GOPATH/bin` directory. (Default | |||
**gokey** can generate passwords and cryptographic private keys (ECC and RSA | |||
keys are currently supported). However, without any additional options | |||
specified it uses your master password as a single source of entropy for | |||
generated data. For passwords it is acceptable most of the time, but keys | |||
generated data. For passwords, it is acceptable most of the time, but keys |
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.
this sounds a bit weird in my head
@@ -8,7 +8,7 @@ | |||
**gokey** is a password manager, which does not require a password vault. | |||
Instead of storing your passwords in a vault it derives your password on the | |||
fly from your master password and supplied _realm_ string (for example, | |||
resource URL). This way you do not have to manage, backup or sync your password | |||
resource URL). This way you do not have to manage, backup, or sync your password |
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.
we already have or
as a separator here - is this an Oxford comma?
@@ -80,7 +80,7 @@ password as a source of entropy. | |||
|
|||
#### Simple mode (without a seed file) | |||
|
|||
In simple mode passwords are derived directly from your master password and the | |||
In simple mode, passwords are derived directly from your master password and the |
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.
same here
Closing as no response |
This README is already well written but I have some suggestions about grammar.