-
Notifications
You must be signed in to change notification settings - Fork 9
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
readme: updates again #29
Conversation
Moves some things around to move usage up and development down a bit. Signed-off-by: Simon de Vlieger <[email protected]>
Include our default project information blurb in the README. Signed-off-by: Simon de Vlieger <[email protected]>
Moves the `test` to `lint`, moves the link into a prettier one. Signed-off-by: Simon de Vlieger <[email protected]>
Last commit needs #28 to be merged to make sense. |
README.md
Outdated
repository including all the tests with: | ||
|
||
``` | ||
make test | ||
make lint |
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.
"including all the tests" is not really true after this change
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.
Changed it back to make test
which now includes pre-commit
again as per #28.
Use `make test` which includes `pre-commit` to conform with the text above it. Signed-off-by: Simon de Vlieger <[email protected]>
Some minor updates to the readme to re-order, make the pre-commit link a bit prettier, and to include our generic project blurb.