Skip to content

Fix custom style file support in workflow#2029

Merged
James Frost (jfrost-mo) merged 1 commit into
mainfrom
1990_colourbar_env_var
Apr 7, 2026
Merged

Fix custom style file support in workflow#2029
James Frost (jfrost-mo) merged 1 commit into
mainfrom
1990_colourbar_env_var

Conversation

@jfrost-mo
Copy link
Copy Markdown
Member

This was broken is two ways. The environment variable wasn't being set in the baking task, so the override wasn't being used, and the quoting was incorrect stopping the CYLC_WORKFLOW_SHARE_DIR expanding.

Fixes #1990

Contribution checklist

Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

  • Documentation has been updated to reflect change.
  • New code has tests, and affected old tests have been updated.
  • All tests and CI checks pass.
  • Ensured the pull request title is descriptive.
  • Ensure rose-suite.conf.example has been updated if new diagnostic added.
  • Conda lock files have been updated if dependencies have changed.
  • Attributed any Generative AI, such as GitHub Copilot, used in this PR.
  • Marked the PR as ready to review.

This was broken is two ways. The environment variable wasn't being set
in the baking task, so the override wasn't being used, and the quoting
was incorrect stopping the CYLC_WORKFLOW_SHARE_DIR expanding.

Fixes #1990
@jfrost-mo James Frost (jfrost-mo) added bug Something isn't working small Small issue that should be quick to resolve labels Apr 7, 2026
@jfrost-mo
Copy link
Copy Markdown
Member Author

Evidence of it working:

Plot with silly colourmap.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 7, 2026

Coverage

@refszkentla
Copy link
Copy Markdown
Contributor

Thanks! You're fast, I was going to do it this week ;)

@jfrost-mo James Frost (jfrost-mo) merged commit ad16a1e into main Apr 7, 2026
7 checks passed
@jfrost-mo James Frost (jfrost-mo) deleted the 1990_colourbar_env_var branch April 7, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working small Small issue that should be quick to resolve

Projects

None yet

Development

Successfully merging this pull request may close these issues.

colorbar env variable not captured in cset_workflow

2 participants