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

Cut down on back-compat warnings, plus other tidying #4943

Merged
merged 7 commits into from
Jun 30, 2022

Conversation

MetRonnie
Copy link
Member

@MetRonnie MetRonnie commented Jun 29, 2022

  • Simplify graph back compat warning and include link to docs Cut down on back compat warnings, on the basis they're not actionable while in back compat mode
  • Replace Task foo: deprecated "host" and "batch system" use "platform" warning with something clearer
  • Clear up cryptic comments - closes Review iso8601.py code comments (and code?) #4805
  • Other small acts of tidying over the course of time

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).
  • Applied any dependency changes to both setup.cfg and conda-environment.yml.
  • Does not need tests
  • No change log entry required
  • No documentation update required.

@MetRonnie MetRonnie added this to the cylc-8.0rc4 milestone Jun 29, 2022
@MetRonnie MetRonnie requested a review from oliver-sanders June 29, 2022 07:24
@MetRonnie MetRonnie self-assigned this Jun 29, 2022
@MetRonnie MetRonnie requested a review from hjoliver June 29, 2022 07:24
@MetRonnie MetRonnie changed the title Tidy Cut down on back-compat warnings, plus other tidying Jun 29, 2022
Comment on lines -354 to -358
REG_CLASH_MSG = (
"The specified reg could refer to ./{0} or ~/cylc-run/{1}. "
"This command will use ./{0}."
)

Copy link
Member Author

Choose a reason for hiding this comment

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

(This was no longer used)

Do not suggest using `allow implicit tasks` setting when rose-suite.conf present in c7 back-compat mode
@oliver-sanders oliver-sanders merged commit 34748a8 into cylc:master Jun 30, 2022
@MetRonnie MetRonnie deleted the tidy branch June 30, 2022 12:59
wxtim added a commit to wxtim/cylc that referenced this pull request Jul 4, 2022
* master: (30 commits)
  Log dir tidy (cylc#4836)
  Cut down on back-compat warnings, plus other tidying (cylc#4943)
  Validate platform settings (background job runner) (cylc#4938)
  clean: push error message to stderr
  Update cylc/flow/id_match.py
  Fix unintended directory stripping during cylc install (cylc#4931)
  stop cylc validate swallowing useful errors (cylc#4936)
  Improve config reference docs (cylc#4916)
  glblcfg: increase default rolling archive length
  Fix job state with pre-submitted failure
  Update tests/functional/cylc-reinstall/02-failures.t
  reinstall: require workflow ID argument
  play: upgrade --start-task's specified in legacy format (cylc#4925)
  expand schema docstring internally (cylc#4926)
  Added a new task filtering test.
  Add comment [skip ci]
  Fix optparse Options class for std options (cylc#4919)
  uid: warn if the run number is provided as a cycle
  Tweak prev.
  Fix task filtering.
  ...
@hjoliver hjoliver modified the milestones: cylc-8.0rc4, cylc-8.0.0 Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Review iso8601.py code comments (and code?)
3 participants