Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #590 +/- ##
==========================================
- Coverage 99.81% 99.06% -0.75%
==========================================
Files 5 5
Lines 1600 1600
==========================================
- Hits 1597 1585 -12
- Misses 3 15 +12 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Issues:
|
d9078b3 to
3923460
Compare
|
@grahamgower @apragsdale -- this is ready for a look. I have shoe-horned most tasks into a place where they now work. The problem is the windows tests. They fail because the modules in demes-spec are not finding ruamel. I cannot reproduce this issue locally, which makes sense because I'm on Linux and the Linux tests are also passing in CI. |
|
The failing tests are in test_cli.py and test_spec.py. What these tests have in common is that they use |
grahamgower
left a comment
There was a problem hiding this comment.
There seems to be a lot of duplication between the _setup.cfg and pyproject.toml - is this intentional? And what's the role of _setup.cfg here, versus setup.cfg? Which files are generated by uv? Is it just uv.lock?
| long_description_content_type = text/markdown | ||
| url = https://github.com/popsim-consortium/demes-python | ||
| classifiers = | ||
| Development Status :: 4 - Beta |
There was a problem hiding this comment.
I reckon we're out of beta now.
| "Topic :: Scientific/Engineering :: Bio-Informatics" | ||
| ] | ||
|
|
||
| requires-python = ">=3.10" |
There was a problem hiding this comment.
This doesn't match the _setup.cfg file.
| readme = {file = "README.md", content-type = "text/markdown"} | ||
| license = {text = "ISC"} | ||
| classifiers = [ | ||
| "Development Status :: 4 - Beta", |
See #589