Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automatically label test merges #89

Merged
merged 1 commit into from
Sep 7, 2024

Conversation

WolfSkin0
Copy link

About The Pull Request

Important

This PR has not been tested comprehensively yet by me because doing that in GitHub actions is a massive pain and I knocked this out in an afternoon between other bits of work.

Runs a cron job that checks TGS against GH for test merges and appropriately (un)labels PRs.

This is a new github actions workflow that runs once every hour and check TGS against GitHub to detect whether a PR has been testmerged or not, applying and removing the 'TestMerge' label as necessary.

We'll need to discuss whether we think having the workflow run once per hour is enough or whether we want to change that.
Aside from that, there's also the case of if we want to have 2-way sync (i.e. applying the label in GitHub causing the PR to get testmerged in TGS automatically.
And there's the question of if we want to fully clean up the code now or focus on just getting this out first and then do cleanup as part of a later cleanup pass for CI.


Changelog

🆑WolfSkin

server: Added auto-labelling workflow for test merges
/:cl:

Runs a cron job that checks TGS against GH for test merges and appropriately (un)labels PRs.
@WolfSkin0 WolfSkin0 added the Server Ops Related to the functional hosting of the server, e.g. TGS stuff label Sep 6, 2024
@WolfSkin0 WolfSkin0 self-assigned this Sep 6, 2024
Copy link

@Takykardi Takykardi left a comment

Choose a reason for hiding this comment

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

Checks out with the rest of the workflows, should be good for deployment.

@valkyria-gk valkyria-gk merged commit 7bb9565 into Equinox-SS13:master Sep 7, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Server Ops Related to the functional hosting of the server, e.g. TGS stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants