Skip to content

Conversation

@forsyth2
Copy link
Collaborator

@forsyth2 forsyth2 commented Oct 9, 2025

Summary

Objectives:

Select one: This pull request is...

  • a bug fix: increment the patch version
  • a small improvement: increment the minor version
  • a new feature: increment the minor version
  • an incompatible (non-backwards compatible) API change: increment the major version

Small Change

  • To merge, I will use "Squash and merge". That is, this change should be a single commit.
  • Logic: I have visually inspected the entire pull request myself.
  • Pre-commit checks: All the pre-commits checks have passed.

@forsyth2 forsyth2 self-assigned this Oct 9, 2025
@forsyth2 forsyth2 added priority: high High priority task (for next release) semver: small improvement Small improvement (will increment patch version) labels Oct 9, 2025
@forsyth2 forsyth2 changed the title Add bug fix to improve robustness and performance Further pcmdi_diags updates Oct 9, 2025
Copy link
Collaborator Author

@forsyth2 forsyth2 left a comment

Choose a reason for hiding this comment

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

I have visually inspected these changes, and tested them as part of the testing noted in E3SM-Project/zppy#742 (review).

@forsyth2 forsyth2 marked this pull request as ready for review October 10, 2025 22:28
@forsyth2
Copy link
Collaborator Author

Merging, per the discussion at E3SM-Project/zppy#742 (comment)

@forsyth2 forsyth2 merged commit d589cee into main Oct 11, 2025
5 checks passed
@forsyth2 forsyth2 deleted the pcmdi-updates branch October 11, 2025 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: high High priority task (for next release) semver: small improvement Small improvement (will increment patch version)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants