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

ci(plotting): use plot hook in prt tests #2092

Merged
merged 2 commits into from
Dec 6, 2024

Conversation

wpbonelli
Copy link
Contributor

@wpbonelli wpbonelli commented Dec 6, 2024

Hook PRT tests up to the mechanism introduced in #2087. Adds some duplication where check and plot functions both need the same data, but IMO it's worth it.


Checklist of items for pull request

  • Replaced section above with description of pull request
  • Referenced issue or pull request ci(plot): add plot option to conftest #2087
  • Added new test or modified an existing test
  • Ran ruff on new and modified python scripts in .doc, autotests, doc, distribution, pymake, and utils subdirectories.
  • Removed checklist items not relevant to this pull request

@wpbonelli wpbonelli added code refactor Nonfunctional changes ci Continuous integration labels Dec 6, 2024
@wpbonelli wpbonelli merged commit f515ca1 into MODFLOW-USGS:develop Dec 6, 2024
20 checks passed
@wpbonelli wpbonelli deleted the prt-plots branch December 6, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration code refactor Nonfunctional changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant