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.

Sign up for GitHub

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

Jump to bottom

job runners: upgrade doc examples to platforms #3997

Closed
oliver-sanders opened this issue Dec 14, 2020 · 2 comments · Fixed by #4006
Closed

job runners: upgrade doc examples to platforms #3997

oliver-sanders opened this issue Dec 14, 2020 · 2 comments · Fixed by #4006
Labels
doc Documentation small
Milestone
cylc-8.0b0

Comments

@oliver-sanders
Copy link
Member

Upgrade documented examples to use platforms syntax.

For example see #3992 (comment)

Pull requests welcome!
@oliver-sanders oliver-sanders added small doc Documentation labels Dec 14, 2020
@oliver-sanders oliver-sanders added this to the cylc-8.0.0 milestone Dec 14, 2020
@oliver-sanders oliver-sanders mentioned this issue Dec 14, 2020
Merged
7 tasks
@wxtim wxtim closed this as completed Dec 17, 2020
@wxtim wxtim reopened this Dec 17, 2020
@wxtim
Copy link
Member

wxtim commented Dec 17, 2020

Misread and closed in error.

@wxtim wxtim mentioned this issue Dec 17, 2020
Merged
2 tasks
@wxtim
Copy link
Member

wxtim commented Dec 17, 2020

AFAIK/Can tell this is actually the only example of this in the job runner handlers. I remember converting them, so I'm not sure why this one slipped the net. Perhaps because there were two examples in one docstring</poor excuse>.

@oliver-sanders oliver-sanders modified the milestones: cylc-8.0.0, cylc-8.0b0 Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
doc Documentation small
Projects
None yet
Milestone
cylc-8.0b0
Development

Successfully merging a pull request may close this issue.

remove job section wxtim/cylc
2 participants
@oliver-sanders @wxtim