Skip to content
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 maintainer docs #511

Merged
merged 9 commits into from Jan 30, 2023
Merged

Update maintainer docs #511

merged 9 commits into from Jan 30, 2023

Conversation

hmgaudecker
Copy link
Collaborator

@hmgaudecker hmgaudecker commented Jan 28, 2023

What problem do you want to solve?

Will fix the second part of #502, i.e., updating the docs on maintaining GETTSIM.

Now that publishing to PyPI worked, I realized that the keywords were in a wrong format, to update them I opened this PR even though publishing to conda-forge is not there yet.

Todo

  • Fix keywords for PyPI.
  • Draft documentation for maintaining using bullets (@timmens)
  • Polish documentation (@hmgaudecker)

@hmgaudecker hmgaudecker marked this pull request as draft January 28, 2023 06:12
@codecov
Copy link

codecov bot commented Jan 28, 2023

Codecov Report

Base: 94.00% // Head: 93.95% // Decreases project coverage by -0.06% ⚠️

Coverage data is based on head (ad3dea2) compared to base (4b7645e).
Patch has no changes to coverable lines.

❗ Current head ad3dea2 differs from pull request most recent head f0f234a. Consider uploading reports for the commit f0f234a to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #511      +/-   ##
==========================================
- Coverage   94.00%   93.95%   -0.06%     
==========================================
  Files          76       76              
  Lines        3788     3788              
==========================================
- Hits         3561     3559       -2     
- Misses        227      229       +2     
Impacted Files Coverage Δ
src/gettsim/__init__.py 90.47% <0.00%> (-9.53%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@hmgaudecker hmgaudecker marked this pull request as ready for review January 30, 2023 10:44
@hmgaudecker hmgaudecker merged commit 175bd44 into main Jan 30, 2023
@hmgaudecker hmgaudecker deleted the maintainer_docs branch January 30, 2023 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants