-
Notifications
You must be signed in to change notification settings - Fork 76
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
chore: Update labeler.yml #13625
chore: Update labeler.yml #13625
Conversation
PR does not require testing, as it is run in the PR checks. |
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! 🚀
Some suggestions, not sure if they are relevant:
- should we also add the labels
backend
/frontend
for changes in the backend/frontend folders? 🤔 ui-editor
is a term that seems to be used only in the labels. I wonder if we should find an English translation closer toutforming
, or maybe use the folder nameux-editor
🤷♂️
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #13625 +/- ##
=======================================
Coverage 94.38% 94.38%
=======================================
Files 1560 1560
Lines 21176 21176
Branches 2529 2529
=======================================
Hits 19986 19986
Misses 946 946
Partials 244 244 ☔ View full report in Codecov by Sentry. |
Sure, that might be relevant wrt who should do reviews etc. Will update!
Yep, although ui-editor is probably more descriptive of what the editor actually is. I'll post a discussion on our slack and we can decide what to use! |
Description
Remove outdated labels
Add missing area labels