Skip to content

Work around coverage issue with netcdf4 library#2009

Open
James Frost (jfrost-mo) wants to merge 1 commit intomainfrom
netcdf4-coverage-workaround
Open

Work around coverage issue with netcdf4 library#2009
James Frost (jfrost-mo) wants to merge 1 commit intomainfrom
netcdf4-coverage-workaround

Conversation

@jfrost-mo
Copy link
Copy Markdown
Member

@jfrost-mo James Frost (jfrost-mo) commented Mar 31, 2026

See conda-forge/netcdf4-feedstock#188

A more permanent workaround than manually reverting each update PR.

Contribution checklist

Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

  • Documentation has been updated to reflect change.
  • New code has tests, and affected old tests have been updated.
  • All tests and CI checks pass.
  • Ensured the pull request title is descriptive.
  • Ensure rose-suite.conf.example has been updated if new diagnostic added.
  • Conda lock files have been updated if dependencies have changed.
  • Attributed any Generative AI, such as GitHub Copilot, used in this PR.
  • Marked the PR as ready to review.

@jfrost-mo James Frost (jfrost-mo) added small Small issue that should be quick to resolve cleanup Non-functional improvement labels Mar 31, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Coverage

@jfrost-mo
Copy link
Copy Markdown
Member Author

James Frost (jfrost-mo) commented Mar 31, 2026

Yep, that workaround works, as shown in #2010.

Copy link
Copy Markdown
Contributor

@jwarner8 James W (jwarner8) left a comment

Choose a reason for hiding this comment

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

Looks good to me, I imagine we'll need to review this occasionally once it is fixed and no longer need to omit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Non-functional improvement small Small issue that should be quick to resolve

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants