Skip to content

Update release script to cope with JULES shared metadata#99

Merged
Pierre Siddall (Pierre-siddall) merged 3 commits into
mainfrom
fix_release
Jul 14, 2025
Merged

Update release script to cope with JULES shared metadata#99
Pierre Siddall (Pierre-siddall) merged 3 commits into
mainfrom
fix_release

Conversation

@jennyhickson

@jennyhickson Jenny Hickson (jennyhickson) commented Jul 9, 2025

Copy link
Copy Markdown
Collaborator

Description

Summary

JULES shared metadata doesn't contain any macros (these are handled seperately) so wants excluding from most of the LFRIc Apps release processes. It does still need to have a copy of the HEAD metadata at the release version.

Changes

  • Change find_meta_dirs() to be more generic, passing in the exclude list seperately.
  • Using find_meta_dirs() to seperately create a set of jules shared metadata directories from the rest and create a combined set of jules shared metadirs and the rest of the metadirs
  • Use the combined set for copy_head_meta() only.

Checklist

  • I have performed a self-review of my own changes

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.

Just a small suggestion

Comment thread lfric_macros/release_lfric.py Outdated
Comment thread lfric_macros/release_lfric.py Outdated

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.

Thanks, looks good

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks good to go to me, well done Jenny merging to main now.

@Pierre-siddall
Pierre Siddall (Pierre-siddall) merged commit c10c876 into main Jul 14, 2025
19 checks passed
@Pierre-siddall
Pierre Siddall (Pierre-siddall) deleted the fix_release branch July 14, 2025 07:30
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.

3 participants