-
Notifications
You must be signed in to change notification settings - Fork 19
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
Uniform dependencies section in suite.rc #22
Uniform dependencies section in suite.rc #22
Conversation
(Will review this after/with cylc/cylc-flow/pull/2935) |
Re-based. |
Companion for cylc/cylc-flow#3191 - cycling and non-cycling suites now have uniform level in the dependencies section.
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.
LGTM 👍
The Cylc Tutorial is in this repo now as well. Do we want to attempt that after #40 or now? |
I guess that's mainly a question for @matthewrmshin. (The order doesn't really matter, except that it affects who does the work). |
Merge both in if possible. Open follow-on issue. |
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.
OK, let's get this show on the road. This one is documentation only, and is just s/old/new/g
, so one review will do.
Companion for cylc/cylc-flow#3191 - cycling and non-cycling suites now have
uniform level in the dependencies section.