-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature #2566 v11.1.0-rc1 #2573
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The compilation step fails because it is setting the DockerHub tag to the string get_dockerhub_tag
.
The push refers to repository [docker.io/library/get_dockerhub_tag]
I made suggested changes that I think should fix it. I tested them in a bash shell locally.
Thanks for catching these issues. Sorry, don’t know how I missed this. I could’ve sworn that it complied fine via GHA but perhaps I just saw the successful documentation workflow run? I’ll get this patched up tomorrow and merge after a successful compilation. |
Re-requesting review. The GHA run compiled fine. It does flag diffs but those are related to recently approved PR's and not this one. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The compilation step is now working properly.
I reviewed the release notes and they look good. All items are descriptive and clear. The links are all correct as well.
I approve.
Expected Differences
Adds release notes for the 11.1.0-rc1 release. Also adds logic to replace '/' with '_' in the Docker tag names used by GHA. This logic was needed for the recent PR #2556 from a fork with a branch named
feature/compile_mac_clang14.x
. The '/' character is not allowed in Docker tag names.Do these changes introduce new tools, command line arguments, or configuration file options? [No]
If yes, please describe:
Do these changes modify the structure of existing or add new output data types (e.g. statistic line types or NetCDF variables)? [No]
If yes, please describe:
Pull Request Testing
Describe testing already performed for these changes:
Confirmed that GHA successfully compiles with the code even with the script changes.
Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:
Please review release notes here
I listed the date as tomorrow (20230614) since it's not getting done today.
Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes]
Do these changes include sufficient testing updates? [Yes]
Will this PR result in changes to the test suite? [No]
If yes, describe the new output and/or changes to the existing output:
Please complete this pull request review by [20230614].
Pull Request Checklist
See the METplus Workflow for details.
Select: Reviewer(s)
Select: Organization level software support Project or Repository level development cycle Project
Select: Milestone as the version that will include these changes