Skip to content

MAPL_GridCompAddSpec: rename default_value -> fill_value#4712

Merged
tclune merged 4 commits into
release/MAPL-v3from
feature/pchakrab/rename-default-value-to-fill-value
Apr 28, 2026
Merged

MAPL_GridCompAddSpec: rename default_value -> fill_value#4712
tclune merged 4 commits into
release/MAPL-v3from
feature/pchakrab/rename-default-value-to-fill-value

Conversation

@pchakraborty
Copy link
Copy Markdown
Contributor

Types of change(s)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Trivial change (affects only documentation or cleanup)
  • Refactor (no functional changes, no api changes)

Checklist

  • Tested this change with a run of GEOSgcm
  • Ran the Unit Tests (make tests)

Description

MAPL_GridCompAddSpec: rename default_value -> fill_value. Also propagate the name change downstream to VariableSpec and state item specs (Field, Bracket, Vector) and affected routines

Related Issue

Also propagate the name change downstream to VariableSpec and state item specs (Field, Bracket, Vector) and affected routines
@pchakraborty pchakraborty self-assigned this Apr 27, 2026
@pchakraborty pchakraborty requested a review from a team as a code owner April 27, 2026 16:59
@pchakraborty pchakraborty added 0 Diff The changes in this pull request have verified to be zero-diff with the target branch. 📈 MAPL3 MAPL 3 Related Changelog Skip Skips the Changelog Enforcer labels Apr 27, 2026
Copy link
Copy Markdown
Collaborator

@tclune tclune left a comment

Choose a reason for hiding this comment

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

You missed some instances in the tests it looks like.

@pchakraborty
Copy link
Copy Markdown
Contributor Author

You missed some instances in the tests it looks like.

I missed a little over 100 files :-(

@tclune tclune merged commit 8e0ba69 into release/MAPL-v3 Apr 28, 2026
39 of 41 checks passed
@pchakraborty pchakraborty deleted the feature/pchakrab/rename-default-value-to-fill-value branch April 28, 2026 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 Diff The changes in this pull request have verified to be zero-diff with the target branch. Changelog Skip Skips the Changelog Enforcer 📈 MAPL3 MAPL 3 Related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants