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

Review names of all settings and in the global and suite configs #3422

Closed
wxtim opened this issue Oct 24, 2019 · 5 comments
Closed

Review names of all settings and in the global and suite configs #3422

wxtim opened this issue Oct 24, 2019 · 5 comments
Labels
superseded

Comments

@wxtim
Copy link
Member

wxtim commented Oct 24, 2019 •
edited by dpmatthews
Loading

As part of the rose suite-run migration work, lots of changes to the global and suite config's were discussed. We've decided to focus on the functionality first and will review all the config settings later (prior to the Cylc 8 release).

We need to look through the discussions in the following issues/PRs:
#3404
#3348
@dpmatthews dpmatthews changed the title [New Config 3] Make simple changes to suite.rc Review names of all settings and in the global and suite configs Nov 19, 2019
@dpmatthews dpmatthews mentioned this issue Nov 19, 2019
Closed
@dpmatthews dpmatthews added this to the cylc-8.0.0 milestone Nov 19, 2019
This was referenced Nov 19, 2019
Closed
Closed
@dpmatthews dpmatthews mentioned this issue Jan 30, 2020
Merged
@dpmatthews
Copy link
Contributor

Changes now agreed: https://cylc.github.io/cylc-admin/proposal-config-changes.html

@hjoliver
Copy link
Member

hjoliver commented May 6, 2020

Can we close this now?

@dpmatthews
Copy link
Contributor

I was assuming we'd leave this issue open until the agreed changes have been implemented.

@hjoliver
Copy link
Member

hjoliver commented May 6, 2020

(The issue title says "review", which is done ... but I'm being pedantic 😬 )

@dpmatthews
Copy link
Contributor

Superceded by:
#3690
#3691
#3692
#3693
#3694
#3695
#3696

@dpmatthews dpmatthews removed this from the cylc-8.0.0 milestone Sep 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
superseded
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dpmatthews @hjoliver @wxtim