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

Use new isodatetime copy methods #4803

Open
1 task
MetRonnie opened this issue Apr 6, 2022 · 0 comments
Open
1 task

Use new isodatetime copy methods #4803

MetRonnie opened this issue Apr 6, 2022 · 0 comments
Assignees
@MetRonnie
Labels
BLOCKED This can't happen until something else does small
Milestone
8.x

Comments

@MetRonnie
Copy link
Member

MetRonnie commented Apr 6, 2022 •
edited
Loading

Needs:

We currently have at least 1 case of mutating isodatetime data classes, however they are supposed to be immutable. We should switch over to the proposed .copy() methods that will allow changing of atttributes. See #4554 (comment)
@MetRonnie MetRonnie added small BLOCKED This can't happen until something else does labels Apr 6, 2022
@MetRonnie MetRonnie added this to the cylc-8.0rc4 milestone Apr 6, 2022
@MetRonnie MetRonnie self-assigned this Apr 6, 2022
@oliver-sanders oliver-sanders modified the milestones: cylc-8.0rc4, cylc-8.x Apr 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MetRonnie MetRonnie

Labels
BLOCKED This can't happen until something else does small
Projects
None yet
Milestone
8.x
Development

No branches or pull requests
2 participants
@oliver-sanders @MetRonnie