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.

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

Move flow.cylc.processed to log dir #4389

Merged
merged 6 commits into from
Sep 2, 2021

Conversation

wxtim
Copy link
Member

@wxtim wxtim commented Sep 2, 2021

These changes close #3763 .

Requirements check-list

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.
  • Contains logically grouped changes (else tidy your branch by rebase).
  • Does not contain off-topic changes (use other PRs for other changes).
  • Already covered by existing tests.
  • Appropriate change log entry included.
  • (master branch) I have opened a documentation PR at move cylc.flow.processed cylc-doc#294

wxtim added 4 commits August 27, 2021 13:34
* 'master' of https://github.com/cylc/cylc:
  GH Actions: Add test workflows badges to release PRs
  Hot fix line length.
  response to review.
  Apply suggestions from code review
  Add unit test.
  update cfgspec doc
  Cycle point format colon restriction
  Codecov: use flags for fast & functional tests
  Remove :cylc:conf:
  Apply suggestions from code review
  Update cylc/flow/cfgspec/globalcfg.py
  Added deprecation warnings in platforms section
  platform documentation changes
  cli: change list_plugins to return a string
  cli: remove gui from command blacklist
  schema: correct faulty enum
CHANGES.md Outdated Show resolved Hide resolved
@MetRonnie MetRonnie added this to the cylc-8.0b3 milestone Sep 2, 2021
Copy link
Member

@hjoliver hjoliver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 tested as working ... I think one review will do, to keep the ball rolling. I'll merge @MetRonnie 's change log suggestion.

Co-authored-by: Ronnie Dutta <[email protected]>
@hjoliver hjoliver merged commit 4c10b6e into cylc:master Sep 2, 2021
@wxtim wxtim deleted the move-processed.rc-to-log-dir branch September 3, 2021 06:42
@MetRonnie MetRonnie changed the title Move processed.rc to log dir Move flow.cylc.processed to log dir Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

future of the suite.rc.processed file
3 participants