-
Notifications
You must be signed in to change notification settings - Fork 94
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
Update change log for 7.8 release #2890
Conversation
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.
Maybe worth mentioning isodatetime has been updated? (From mobile, sorry shot text/review). Rest looks good to me
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.
Quick comments.
I don't normally mention isodatetime updates, treating it as an internal library that users hopefully don't need to know about (happy to change if others prefer though). |
(No preference on isodatetime - it is not really relevant to users - so happy for it to be omitted.) |
4e82ac9
to
7ef4824
Compare
Thanks for the feedback @kinow and @matthewrmshin - I'm going ahead with this. "Time is of the essence" ... and I don't want to have to eat my 🎩 |
@cylc/core - anyone want to take a quick look at this, to see if I missed anything important?