Skip to content
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

Base PR-Based Repro-Testing Workflow #136

Merged
merged 8 commits into from
Oct 17, 2024
Merged

Base PR-Based Repro-Testing Workflow #136

merged 8 commits into from
Oct 17, 2024

Conversation

CodeGat
Copy link
Contributor

@CodeGat CodeGat commented Oct 15, 2024

This PR is a slimmed-down variant of the now-closed #122 PR.

This PR contains only the base workflow as specified in access-nri/model-configs-template, similar to the existing access-[om2|esm1.5]-configs repositories. This is so we can merge in functionality that is set in stone, with additions that require extra time/thought to come later.

References #120

@CodeGat CodeGat added CI Continuous integration infrastructure priority:high labels Oct 15, 2024
@CodeGat CodeGat self-assigned this Oct 15, 2024
Co-authored-by: Anton Steketee <[email protected]>
@anton-seaice
Copy link
Contributor

This looks good Tommy - is it ready to go ?

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@CodeGat
Copy link
Contributor Author

CodeGat commented Oct 15, 2024

Hey @anton-seaice - it is ready to go. It's the base PR workflow shared between all the other access-*-configs repos we have, so it doesn't yet have the docs bit that you require. I still think it's worth incrementally adding the bits that are set in stone, just so we don't have a 100-year PR while other bits are being hashed out.

There is a follow-up PR here for adding the !test comment command so you can test/commit checksums in non-release-* branches, I'm just testing it in my own org first.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
Copy link
Contributor

@anton-seaice anton-seaice left a comment

Choose a reason for hiding this comment

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

Thanks Tommy!

@CodeGat CodeGat merged commit 190d914 into main Oct 17, 2024
@CodeGat CodeGat deleted the 120-base-pr-workflow branch October 17, 2024 02:24
CodeGat added a commit that referenced this pull request Oct 18, 2024
CodeGat added a commit that referenced this pull request Oct 18, 2024
CodeGat added a commit that referenced this pull request Oct 18, 2024
CodeGat added a commit that referenced this pull request Oct 18, 2024
CodeGat added a commit that referenced this pull request Oct 18, 2024
* Migrated CI from #137 and #136

* Quoted "!test" job name
CodeGat added a commit that referenced this pull request Oct 24, 2024
* Migrated CI from #137 and #136

* Quoted "!test" job name
CodeGat added a commit that referenced this pull request Oct 24, 2024
* Migrated CI from #137 and #136

* Quoted "!test" job name
CodeGat added a commit that referenced this pull request Oct 24, 2024
* Migrated CI from #137 and #136

* Quoted "!test" job name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration infrastructure priority:high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants