Skip to content

Conversation

@jennyhickson
Copy link
Contributor

Currently, once the code reviewer requests changes or approves a PR the logic to recognise a trivial PR breaks. Add checks for these states to the workflow.

Copy link
Contributor

@james-bruten-mo james-bruten-mo left a comment

Choose a reason for hiding this comment

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

A missing $ but otherwise looks good

Co-authored-by: James Bruten <109733895+james-bruten-mo@users.noreply.github.com>
Copy link
Contributor

@james-bruten-mo james-bruten-mo left a comment

Choose a reason for hiding this comment

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

Thanks

@james-bruten-mo james-bruten-mo merged commit 6f12c3c into MetOffice:main Jan 23, 2026
2 checks passed
@jennyhickson jennyhickson added this to the Spring 2026 milestone Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants