-
Notifications
You must be signed in to change notification settings - Fork 23
Conversation
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.
Nice start on the README! Feels a bit incomplete, feels like we can improve the developer experience a bit. Check out my comments.
To the outside world, we provide an API, not a server.
- Add how to extract and add to the PATH - Add how to install from source
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.
It's getting better and better! I have a few additions and tweaks to further improve the developer experience.
I think this README.md can be one of our most important READMEs of all, as it is the starting point for getting started with SecretHub. Therefore we should treat it as a real landing page and give it all the goodies it deserves.
This reverts commit 02b9151.
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.
Looks good! Few minor comments, feel free to merge when satisfied.
Now the invite comes from SecretHub instead of Simon Barendse
No description provided.