-
Notifications
You must be signed in to change notification settings - Fork 15
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
ci: add workflow for PR automation #319
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work 🙂 Good idea to add the "testing" and "CI/CD" labels, too 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I created an example PR and tried it out, nice QoL improvements! 🎉
The author self-assignment works. ✔️
The PR title commit lint also works well. ✔️
The only issue I see is the recognition of testing/production code/CI paths which were already mentioned by Matěj. 🎨
86b6d06
to
9cf7c63
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice automations!
Part of #142
plugin-eslint
), the corresponding label should be assignedMake three integrations. Another has to be done in another PR.