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

Re-cookiecut: implement codecov, install pdffit2 rc, limit to py3.12 #235

Merged
merged 9 commits into from
Dec 7, 2024

Conversation

bobleesj
Copy link
Contributor

@bobleesj bobleesj commented Dec 7, 2024

Closes #232 - add codecov report in PR
Closes #231 - implement codespell
Closes #230 - add requires-python in pyproject.toml
Closes #236 - fix spelling suggested by Codespell

Copy link

codecov bot commented Dec 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.11%. Comparing base (9177637) to head (f1c37df).
Report is 16 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #235       +/-   ##
===========================================
+ Coverage   55.86%   95.11%   +39.25%     
===========================================
  Files          96       23       -73     
  Lines       11202     1126    -10076     
===========================================
- Hits         6258     1071     -5187     
+ Misses       4944       55     -4889     

see 89 files with indirect coverage changes

run: |
conda install --file requirements/test.txt
conda install wxpython diffpy.utils matplotlib-base
pip install diffpy.pdffit2==1.4.4rc6
Copy link
Contributor Author

@bobleesj bobleesj Dec 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we are testing pdfgui with pdffit rc, we conda install wxpython, diffpy.utils, and matplotlib-base

and then we pip install pdffit2 rc

Once we have pdfgui tested with pdffit2 rc, we could try to find an approach to use the reusable workflow? @sbillinge

@sbillinge sbillinge merged commit dd9f989 into diffpy:main Dec 7, 2024
5 checks passed
@sbillinge
Copy link
Contributor

Nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants