Skip to content

Cookiecutter update, GIS testing adjustments#589

Merged
Zeitsperre merged 15 commits intomainfrom
cookiecutter-update
Mar 16, 2026
Merged

Cookiecutter update, GIS testing adjustments#589
Zeitsperre merged 15 commits intomainfrom
cookiecutter-update

Conversation

@Zeitsperre
Copy link
Member

@Zeitsperre Zeitsperre commented Mar 12, 2026

Pull Request Checklist:

  • This PR addresses an already opened issue (for bug fixes / features)
    • This PR fixes #xyz
  • (If applicable) Documentation has been added / updated (for bug fixes / features).
  • (If applicable) Tests have been added.
  • CHANGELOG.rst has been updated (with summary of main changes).
    • Link to issue (:issue:number) and pull request (:pull:number) has been added.

What kind of change does this PR introduce?

  • Updates the cookiecutter to the latest version
    • Enables PEP 735 (dependency-groups)
    • Replaces pre-commit with prek
    • The Makefile now handles calls to install dependencies from tox
  • Tests have been adjusted to allow for testing without GIS libraries.

Does this PR introduce a breaking change?

Yes. The dev and docs install recipes are no longer available from the sdist and wheel distributions. These recipes are specifically for developers and are now only accessible when examining the source code. ravenpy[gis,raven-hydro] remain as valid install recipes.

Other information:

https://peps.python.org/pep-0735/
https://prek.j178.dev/

Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
@github-actions
Copy link
Contributor

Note

It appears that this Pull Request modifies the main.yml workflow.

On inspection, the RAVEN_TESTDATA_BRANCH environment variable is set to the most recent tag (v2025.6.12).

No further action is required.

Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Copy link
Collaborator

@huard huard left a comment

Choose a reason for hiding this comment

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

Thanks Trevor for the clean-up, much appreciated.

I'll just want to confirm with James that changes in outputs are expected from the Raven update.

Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
@Zeitsperre Zeitsperre merged commit 62420a0 into main Mar 16, 2026
26 checks passed
@Zeitsperre Zeitsperre deleted the cookiecutter-update branch March 16, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants