-
Notifications
You must be signed in to change notification settings - Fork 51
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
Rephrase cycle time to cycle point #1323
Conversation
Fix tests for cylc 6. When checking whether a suite is running or not, allow reasons to contain all lines of the `pgrep` command.
Simply return `TimePoint` object if format string is `None`.
@benfitzpatrick please review. |
if "%" in print_format: | ||
if print_format is None: | ||
return str(time_point) | ||
if print_format and "%" in print_format: |
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.
First part of the if
isn't necessary here, I think.
We can now assume that cylc stores date time correctly.
Fixed unnecessary I have also fixed some unnecessary Z syntax in the Rose Bush templates. This is to assume that cylc is now storing date time with time zone information. |
This is always supposed to be true now. |
Rephrase cycle time to cycle point
Rephrase cycle time to cycle point for cylc 6.
Improve reporting of running cylc processes.
Now that cylc/cylc-flow#996 is closed, we need this for Rose to work correctly.