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

Update labeler config #1934

Merged
merged 3 commits into from
Jan 24, 2023
Merged

Update labeler config #1934

merged 3 commits into from
Jan 24, 2023

Conversation

kachkaev
Copy link
Contributor

@kachkaev kachkaev commented Jan 24, 2023

🌟 What is the purpose of this PR?

The aim of this PR is to re-enable Slack notification for changes related to HASH app. It replaces a few globs for packages/hash/... with apps/hash-*/**, libs/hash-*/** and libs/@local/hash-*/**. To avoid repetition, I replaced the official labeler versio with the latest commit from actions/labeler#316.

🔗 Related links

🐾 Next steps

  • Check if labeler works (we can only see this once the PR is merged because we use pull_request_target as a trigger)
  • Wait for dot option to reach GA and use the official labeler.
  • We can also consider syncing the labels with BP repo and get rid of A/B/C-. This does not bring a lot of added value in short term, so I haven’t tasked that out.

🛡 What tests cover this?

@github-actions github-actions bot added the area/infra Relates to version control, CI, CD or IaC (area) label Jan 24, 2023
@kachkaev
Copy link
Contributor Author

/trunk merge

@trunk-io trunk-io bot merged commit d59c192 into main Jan 24, 2023
@trunk-io trunk-io bot deleted the ak/update-labeler-config branch January 24, 2023 18:53
@trunk-io
Copy link

trunk-io bot commented Jan 24, 2023

😎 Merged successfully (details).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/infra Relates to version control, CI, CD or IaC (area)
Development

Successfully merging this pull request may close these issues.

2 participants