Install shumlib and mule in unit tests action#118
Merged
arjclark merged 20 commits intoMetOffice:mainfrom Feb 24, 2026
Merged
Conversation
This reverts commit 7c69899.
mo-tgeddes
previously requested changes
Feb 19, 2026
Collaborator
mo-tgeddes
left a comment
There was a problem hiding this comment.
This looks good thank you. Can you please add the changes to the changelog?
Collaborator
Author
Created #121 for adding the changelog |
arjclark
approved these changes
Feb 24, 2026
Theo is away - request captured in separate ticket (#121)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #117
To be completed prior to review request and updated as required during the review process.
If the answer to an item on the list is not applicable, feel free to replace the checkbox with 'N/A' to give extra clarity.
All developers are reminded to follow the ancil working practices
Branch
Related branches (e.g. contrib):
N/A
ANTS rose stem logs
ANTS/run11
contrib rose stem logs
N/A
Testing
For core ANTS only tests, the bare minimum that will be accepted is the
--group=unittestsbut many, if not most, changes will need to test other groups to ensure they meet reviewer expectations. In general, it should be possible and is advised to run the--group=allgroup prior to review submission as this will catch any consequential issues. Additionally you must run the contrib tests, pointing at your branch, with--group=allto capture any behaviour changes affecting Science codes.If your change will alter existing science results, you will need to seek appropriate Scientific validation and confirm that the model has been initialised with your new development. Inspecting a change in xconv/pyplot/visualiser of choice is not sufficient to demonstrate the model can be initialised from your file.
Impact of change
rose stem --group=alltestsrose stem --group=alltestsApprovals for this change
New functionality further testing
These tasks must succeed for your ticket to pass review.
Other
Rose stem logs
Please copy in the contents of your trac_status.log file(s) below (found in the cylc-run directory for your rose stem run) to your rose-stem testing here. Note: if your changes lead to a change in answers, you must run
rose stem --group=allto help ensure all affected configurations has been flagged up.Test Results - Summary
Test Results - Detail