Skip to content

Remove unused rose configuration settings#2226

Merged
James Frost (jfrost-mo) merged 2 commits into
mainfrom
remove_unused_config_settings
Jun 25, 2026
Merged

Remove unused rose configuration settings#2226
James Frost (jfrost-mo) merged 2 commits into
mainfrom
remove_unused_config_settings

Conversation

@jfrost-mo

@jfrost-mo James Frost (jfrost-mo) commented Jun 24, 2026

Copy link
Copy Markdown
Member

This allows the configurations to validate cleanly, as needed for #2227.

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.
  • Attributed any Generative AI, such as GitHub Copilot, used in this PR.
  • Marked the PR as ready to review.

@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Coverage

@JorgeBornemann

Copy link
Copy Markdown
Collaborator

All good, ready to merge.

@jfrost-mo

Copy link
Copy Markdown
Member Author

I'm going to unstack this PR and merge it, as it doesn't require #2225.

This allows them to validate cleanly.
This allows it to validate cleanly.
@jfrost-mo James Frost (jfrost-mo) force-pushed the remove_unused_config_settings branch from fc16fd6 to 564daff Compare June 25, 2026 08:48
@jfrost-mo James Frost (jfrost-mo) changed the base branch from remove_qq_include to main June 25, 2026 08:51
@jfrost-mo James Frost (jfrost-mo) merged commit 564daff into main Jun 25, 2026
7 checks passed
@jfrost-mo James Frost (jfrost-mo) deleted the remove_unused_config_settings branch June 25, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Non-functional improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants