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

8.0.x -> master #5206

Merged
merged 13 commits into from
Oct 25, 2022
Merged

8.0.x -> master #5206

merged 13 commits into from
Oct 25, 2022

Conversation

MetRonnie
Copy link
Member

No description provided.

oliver-sanders and others added 13 commits October 13, 2022 16:06
* Selectors were not correctly preserved for the `task` and
  `workflow` properties of `Tokens` objects.
* This effectively stripped cycle/task selectors from
  IDs passed to `cylc trigger`.
* Closes cylc#5186
uid: include selectors in detokenised IDs where required

* Add a new interface for retrieving the string ID including selectors from `Tokens` objects.
* Use this for CLI scripts which utilise selectors (e.g. `kill`).
* Fixes a bug where selectors provided on the CLI were silently stripped before being passed through GraphQL.
id: preserve selectors for "task" and "workflow" properties
Workflow: Release stage 1 - create release PR (Cylc 8+ only), run: 15

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Workflow: Release stage 2 - auto publish, run: 35

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…_Scan_Traceback_Bug

fix an error handling bug in Cylc Scan
@MetRonnie MetRonnie added the sync PR for synchronising changes between 8.*.x and master label Oct 25, 2022
@MetRonnie MetRonnie added this to the cylc-8.1.0 milestone Oct 25, 2022
@MetRonnie MetRonnie self-assigned this Oct 25, 2022
@oliver-sanders oliver-sanders requested review from wxtim and removed request for oliver-sanders October 25, 2022 09:58
@wxtim wxtim merged commit b66c856 into cylc:master Oct 25, 2022
@MetRonnie MetRonnie deleted the 8.0.x-deconflict branch October 25, 2022 10:47
@MetRonnie MetRonnie restored the 8.0.x-deconflict branch October 25, 2022 10:50
@MetRonnie MetRonnie deleted the 8.0.x-deconflict branch October 25, 2022 12:51
wxtim added a commit to wxtim/cylc that referenced this pull request Oct 31, 2022
… of github.com:wxtim/cylc into 20221010T0945--HEAD--get_template_vars_from_running_wf

* '20221010T0945--HEAD--get_template_vars_from_running_wf' of github.com:wxtim/cylc: (51 commits)
  make revalidate automatic if workflow has changed at source
  play: --upgrade and --downgrade options
  Fix consolidation tutorial (cylc#5199)
  8.0.x -> master (cylc#5206)
  GH Actions Codecov upload: try using token arg (cylc#5179)
  Update CHANGES.md [skip ci]
  Add type hints.
  Switch to better monkeypatching approach. [skip ci]
  Add explanatory comments. [skip ci]
  Update CHANGES.md [skip ci]
  play: check Cylc version on restart
  Use a context manager in new test.
  Update change log.
  Extend new test coverage.
  install: scan only target workflow
  Apply suggestions from code review
  Add integration test.
  Scan workflow name during install.
  fix an error handling bug in Cylc Scan
  update PR template (cylc#5202)
  ...
@MetRonnie MetRonnie added the duplicate This is a duplicate of something else label Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This is a duplicate of something else sync PR for synchronising changes between 8.*.x and master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants