-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add a pypi badge #54
Add a pypi badge #54
Conversation
@@ -9,7 +9,7 @@ authors: | |||
orcid: "https://orcid.org/0000-0000-0000-0000" |
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.
Please add Yang's and your information to this file, too.
CITATION.cff
Outdated
@@ -9,7 +9,7 @@ authors: | |||
orcid: "https://orcid.org/0000-0000-0000-0000" | |||
date-released: 20??-MM-DD |
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.
Let's add the date in this PR too, e.g. 2022-11-24 or 25
@@ -60,7 +61,7 @@ conda environment before submitting the the bash script. See | |||
### On CRIB | |||
|
|||
[CRIB](https://crib.utwente.nl/) is the ITC Geospatial Computing Platform. You | |||
can run the model using `Matlab` or `Octave`. Currently, running the | |||
can run the model using `Matlab` or `Octave`. Currently, running the | |||
exceutable file on CRIB is not supported because MATLAB Runtime can not be | |||
installed there. See [Install PyStemmusScope](#install-pystemmusscope). |
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.
There are two commands below that uses pip install
from main
branch. when pystemmusscope is published on pypi, we can install it by pip install pystemmusscope
. Shall we fix the commands in this PR?
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.
Done!
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.
@BSchilperoort thanks for the changes and fixing the linter 👍 please see my comments.
Kudos, SonarCloud Quality Gate passed! |
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.
@BSchilperoort many thanks.
Thanks a lot, Sarah, Bart, Yang, this is cool! |
This PR adds the pypi version badge to the README.md.
The version has been bumped to 0.1.1, this will allow us to properly test the Action that pushes the new releases to pypi.
pyroma
(setup.cfg missing "python_requires" in[options]
).Closes #52