Skip to content

Cookiecutter update, GIS testing adjustments#589

Open
Zeitsperre wants to merge 10 commits intomainfrom
cookiecutter-update
Open

Cookiecutter update, GIS testing adjustments#589
Zeitsperre wants to merge 10 commits intomainfrom
cookiecutter-update

Conversation

@Zeitsperre
Copy link
Member

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,all] 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>
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.

1 participant