-
Notifications
You must be signed in to change notification settings - Fork 11
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
Plugin fails if CLI opts given, but workflow not a Rose suite. #118
Conversation
2319cb5
to
cb4c333
Compare
I've accidentally discovered that this is broken - converted to draft until I've fixed. |
And now fixèd. |
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.
Code read, tests run locally and manually tested - no problems found. Currently waiting for the cylc and rose test batteries to run to double check for any breakages.
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.
All working well. Thanks @wxtim.
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.
Looks good, couple of suggestions
Co-authored-by: Ronnie Dutta <61982285+MetRonnie@users.noreply.github.com>
Closes #117
@dpmatthews - I'm tagging you here, so you get a message that I've put a PR up, but not list you as a reviewer.