Skip to content

Conversation

@QSparks
Copy link
Contributor

@QSparks QSparks commented Jun 13, 2025

  • PEP 621-conformant pyproject.toml
  • Poetry >2.0
  • Separate Black CI workflow
  • Use a 3.11-slim for docker images
  • Docs for running PoethePoet
  • Use marble-dev01 URLs
  • Replace pkg_resources with importlib.resources
  • Fix file references

Pending:

  • Use tagged wps-tools version once merged
  • Use tagged nchelpers version once merged

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@QSparks QSparks marked this pull request as ready for review June 17, 2025 18:49
Copy link

@Nospamas Nospamas left a comment

Choose a reason for hiding this comment

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

Tough one to review. Lots of generated code, don't see anything that stands out as wrong. LGTM

@QSparks QSparks merged commit 4948914 into master Jun 20, 2025
4 checks passed
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.

3 participants