-
Notifications
You must be signed in to change notification settings - Fork 12
Update the Readme #301
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 the Readme #301
Conversation
jpaillard
left a comment
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.
I have a personal preference for a more streamlined Readme (see skrub for instance).
But please feel free to ignore my comments if you like a more thorough one.
|
|
||
| .. # Add the reference for the badges | ||
| .. ## Reference to the CI status | ||
| .. |Linter&Tests| image:: https://github.com/mind-inria/hidimstat/actions/workflows/ci.yml/badge.svg?branch=main |
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 badge does not render well
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 normal because the name of the workflow is not yet modified.
It will be done with this PR.
bthirion
left a comment
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.
LGTM, thx.
This is an update of the readme.
For see the result, follow this link: https://github.com/lionelkusch/hidimstat/tree/PR_README.txt
For updating the README, I get inspiration from the following repository: