Skip to content

Metadata: relative_path --> dependencies_path#693

Merged
mkavulich merged 2 commits into
NCAR:developfrom
climbfuji:feature/rel_path_dep_path
Dec 4, 2025
Merged

Metadata: relative_path --> dependencies_path#693
mkavulich merged 2 commits into
NCAR:developfrom
climbfuji:feature/rel_path_dep_path

Conversation

@climbfuji

Copy link
Copy Markdown
Collaborator

Metadata: relative_path --> dependencies_path

Change option relative_path to dependencies_path as discussed in #685

User interface changes?: Yes
CCPP metadata: relative_path --> dependencies_path

Fixes #685

Testing:
test removed: n/a
unit tests:
system tests:
manual testing:

@dustinswales dustinswales left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One small, possibly irrelevant comment.

Comment thread scripts/metadata_table.py

@gold2718 gold2718 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I did not look really hard for any gotchas but this looks pretty clean.

@climbfuji

Copy link
Copy Markdown
Collaborator Author

@mkavulich This is ready to merge, has approvals from all partners

@mkavulich mkavulich merged commit 56260e2 into NCAR:develop Dec 4, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants