Skip to content

Remove unused QQ plot cylc include file#2225

Merged
James Frost (jfrost-mo) merged 1 commit into
mainfrom
remove_qq_include
Jun 29, 2026
Merged

Remove unused QQ plot cylc include file#2225
James Frost (jfrost-mo) merged 1 commit into
mainfrom
remove_qq_include

Conversation

@jfrost-mo

Copy link
Copy Markdown
Member

It was never removed after QQ plots where ported to loaders, despite never being used.

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.

@jfrost-mo James Frost (jfrost-mo) added small Small issue that should be quick to resolve cleanup Non-functional improvement labels Jun 24, 2026
@github-actions

github-actions Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Coverage

@daflack David Flack (daflack) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good spot.

It was never removed after QQ plots where ported to loaders, despite
never being used.
@jfrost-mo James Frost (jfrost-mo) merged commit e1a3d24 into main Jun 29, 2026
8 checks passed
@jfrost-mo James Frost (jfrost-mo) deleted the remove_qq_include branch June 29, 2026 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Non-functional improvement small Small issue that should be quick to resolve

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants