-
Notifications
You must be signed in to change notification settings - Fork 8
Correct/Update CI #106
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
Correct/Update CI #106
Conversation
You can also create the test package on test.PyPi.org — we do this for most of our packages. This will allow you to check everything. |
97918ce
to
ec5b1fb
Compare
* remove py3.8 * try fixing codecov * move setup.py into pyproject.toml
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #106 +/- ##
==========================================
- Coverage 83.42% 83.06% -0.37%
==========================================
Files 8 8
Lines 555 555
Branches 116 116
==========================================
- Hits 463 461 -2
- Misses 75 77 +2
Partials 17 17 ☔ View full report in Codecov by Sentry. |
@PicoCentauri , I see you want to move to |
Thanks for the link. I will update our files accordingly. |
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 am happy with the new structure. If you @joaomcteixeira also agree we can merge this.
Closes #105, #107
The problem is that
long_description
insetup.cfg
can only accept two files. Or at least setting two is does not work the same as fortoml
syntax.To see if it works:
Go to
dist/
open the.whl
file and see theMETADATA
file. The README description for PyPI is there.