Skip to content

Conversation

@climbfuji
Copy link
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:

Copy link
Member

@dustinswales dustinswales left a comment

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.

Copy link
Collaborator

@gold2718 gold2718 left a comment

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
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