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

Introduce Probot Autolabeler #11363

Merged
merged 1 commit into from
Apr 16, 2020
Merged

Conversation

Zales0123
Copy link
Member

Q A
Branch? 1.6
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

As a General Hater of Not-labeled Pull Requests, to help our work easier, I would like to introduce a https://github.com/apps/probot-autolabeler.

I'm not sure if it's the best choice from auto labelling bots, but it has one big advantage - can label PRs from forked repositories (which is crucial for Sylius). It has also a very simple configuration (which is both ➕ and ➖), so maybe we would have to modify it (or replace with something better) to handle some more sophisticated algorithms for labels 🏇

This is just a first version of configuration - when it's merged, I would love to see some more lines there (for example for API label on master branch 💃)

@Zales0123 Zales0123 added the Maintenance CI configurations, READMEs, releases, etc. label Apr 16, 2020
@Zales0123 Zales0123 requested a review from a team as a code owner April 16, 2020 10:52
@Zales0123 Zales0123 changed the base branch from master to 1.6 April 16, 2020 10:52
@Zales0123 Zales0123 force-pushed the introduce-autolabeler branch from c45d38c to dbcc105 Compare April 16, 2020 10:56
Copy link
Contributor

@pamil pamil left a comment

Choose a reason for hiding this comment

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

Looks safe to try!

@pamil pamil merged commit a71b3a4 into Sylius:1.6 Apr 16, 2020
@pamil
Copy link
Contributor

pamil commented Apr 16, 2020

Thank you, Mateusz! 🥇

@Zales0123 Zales0123 deleted the introduce-autolabeler branch April 17, 2020 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance CI configurations, READMEs, releases, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants