Skip to content

Fixing tutorials due to an order change which didn't quite work.#7289

Open
samuel-denton wants to merge 8 commits into
cylc:8.6.xfrom
samuel-denton:fix_config_consolidation_tutorials
Open

Fixing tutorials due to an order change which didn't quite work.#7289
samuel-denton wants to merge 8 commits into
cylc:8.6.xfrom
samuel-denton:fix_config_consolidation_tutorials

Conversation

@samuel-denton
Copy link
Copy Markdown
Contributor

@samuel-denton samuel-denton commented Apr 28, 2026

Relates to cylc/cylc-doc#924 but doesn't close it until cylc/cylc-doc#909 is also merged.
Reverted #7238 as order of tutorials has changed again.
Will also need to slightly re-do the JinJa2 and Parameters tutorials to work in the new (old) order.

Check List

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.
  • Contains logically grouped changes (else tidy your branch by rebase).
  • Does not contain off-topic changes (use other PRs for other changes).
  • Applied any dependency changes to both setup.cfg (and conda-environment.yml if present).
  • Tests are included (or explain why tests are not needed).
  • Changelog entry included if this is a change that can affect users
  • Cylc-Doc pull request opened if required at cylc/cylc-doc/pull/XXXX.
  • If this is a bug fix, PR should be raised against the relevant ?.?.x branch.

@samuel-denton samuel-denton self-assigned this Apr 28, 2026
@samuel-denton samuel-denton added doc Documentation could be better Not exactly a bug, but not ideal. labels Apr 28, 2026
@samuel-denton samuel-denton added this to the 8.6.x milestone Apr 28, 2026
@samuel-denton samuel-denton marked this pull request as ready for review April 30, 2026 16:22
@samuel-denton
Copy link
Copy Markdown
Contributor Author

samuel-denton commented Apr 30, 2026

I have run through the tutorials multiple times and I think I am now happy with the order and the accompanying text.

Please review alongside cylc/cylc-doc#909 and close cylc/cylc-doc#924

removed Jinja2 shebang as it is added by the next tutorial.
Im stupid, this was needed for the includes statement at the end...
@MetRonnie MetRonnie modified the milestones: 8.6.4, 8.6.5 May 12, 2026
@samuel-denton samuel-denton marked this pull request as draft May 13, 2026 09:43
@samuel-denton samuel-denton marked this pull request as ready for review May 18, 2026 16:33
@oliver-sanders oliver-sanders requested review from MetRonnie and removed request for ChrisPaulBennett May 20, 2026 13:29
Comment thread cylc/flow/etc/tutorial/cylc-forecasting-workflow/flow.cylc Outdated
Comment thread cylc/flow/etc/tutorial/consolidation-tutorial/flow.cylc Outdated
samuel-denton and others added 2 commits May 22, 2026 12:12
Co-authored-by: Ronnie Dutta <61982285+MetRonnie@users.noreply.github.com>
@samuel-denton samuel-denton requested a review from MetRonnie May 22, 2026 11:16
Comment thread cylc/flow/etc/tutorial/cylc-forecasting-workflow/flow.cylc Outdated
Co-authored-by: Ronnie Dutta <61982285+MetRonnie@users.noreply.github.com>
@MetRonnie MetRonnie self-requested a review May 27, 2026 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

could be better Not exactly a bug, but not ideal. doc Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants