Skip to content

Action to move PRs through project state - #52

Merged
Yaswant Pradhan (yaswant) merged 10 commits into
MetOffice:mainfrom
james-bruten-mo:project_edit_action
Jan 14, 2026
Merged

Action to move PRs through project state#52
Yaswant Pradhan (yaswant) merged 10 commits into
MetOffice:mainfrom
james-bruten-mo:project_edit_action

Conversation

@james-bruten-mo

@james-bruten-mo James Bruten (james-bruten-mo) commented Jan 7, 2026

Copy link
Copy Markdown
Collaborator

This reusable workflow adds the ability to edit the Simulation Systems Review Tracker project.
Based on the reviewer entries in the PR template it will fill in the reviewer fields in the tracker.
Based on completed reviews by the SR and CR it will modify the project state.
It automatically adds the PR author as an assignee.
It will request a review from the CR if they are not already added.

To edit projects requires a PAT with project write access to be added as a secret to each calling repository. I propose we add a PAT to the umadmin account with that permission and use this as the secret for each repo.

There's a PR in the git_playground demonstrating it working https://github.com/MetOffice/git_playground/pull/112

We probably also want to check it works on the git_playground as a reusable workflow as well.

@james-bruten-mo

Copy link
Copy Markdown
Collaborator Author

I'm happy with the testing of this now - it's works from forks for all required triggers. Tagging Yaswant Pradhan (@yaswant) for a review for this going onto main

@yaswant Yaswant Pradhan (yaswant) 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.

We have done enough testing, and identified areas of improvements. I'm happy for this to go as an initial implementation.

@james-bruten-mo

Copy link
Copy Markdown
Collaborator Author

Thanks Yash, I've asked Jenny Hickson (@jennyhickson) to review the PRs adding the calling workflows, so passing this to her for CR too

@yaswant
Yaswant Pradhan (yaswant) merged commit f91d325 into MetOffice:main Jan 14, 2026
2 checks passed
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.

3 participants