Validate cylc workflow and rose metadata via automatic tests#2227
Draft
James Frost (jfrost-mo) wants to merge 5 commits into
Draft
Validate cylc workflow and rose metadata via automatic tests#2227James Frost (jfrost-mo) wants to merge 5 commits into
James Frost (jfrost-mo) wants to merge 5 commits into
Conversation
James Frost (jfrost-mo)
added
enhancement
James Frost (jfrost-mo)
changed the title
7 tasks
Contributor
|
|
James Frost (jfrost-mo)
requested review from
JorgeBornemann (JorgeBornemann)
and removed request for
JorgeBornemann (JorgeBornemann)
Member
Author
|
Ah, I'll see about getting cylc/setup-cylc fixed so we can use it; it needs to pin the actions it is using itself. I'll put this PR on hold until then. |
James Frost (jfrost-mo)
force-pushed
the
remove_unused_config_settings
branch
from
fc16fd6 to
564daff
Compare
This uses a very minimal configuration to act as a smoke test that new configuration values are correctly made optional or added to the example configuration.
This avoids needing cylc installed on our generic test runners.
James Frost (jfrost-mo)
force-pushed
the
310_test_workflow
branch
from
58c63e8 to
530a3b5
Compare
James Frost (jfrost-mo)
force-pushed
the
310_test_workflow
branch
from
530a3b5 to
fc62c1f
Compare
James Frost (jfrost-mo)
force-pushed
the
310_test_workflow
branch
from
fc62c1f to
91504c0
Compare
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
1 participant
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.
Part of #310, but doesn't quite do enough to call the issue fixed.
Contribution checklist
Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.
rose-suite.conf.examplehas been updated if new diagnostic added.