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.

Sign up for GitHub

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(ci): Labeler needs updating after Learn move #37294

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
+67 −75
Open

chore(ci): Labeler needs updating after Learn move #37294

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b06fa20
chore(ci): Update labeler following learn move, make matching coarser
bsmth Dec 20, 2024
b328ad5
chore(ci): Update labeler following learn move, make matching coarser
bsmth Dec 20, 2024
efc8181
chore(ci): Update labeler following learn move, make matching coarser
bsmth Dec 20, 2024
2ece8d5
Merge branch 'main' into labeler-rename
bsmth Jan 14, 2025
ff4610e
chore: restore deleted directory configs as comment
bsmth Jan 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 1 addition & 21 deletions .github/issue-regex-labeler.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,27 +19,7 @@ Content:HTTP:
Content:JS:
- '\/docs\/Web\/JavaScript'
Content:Learn:
Copy link
Member

Choose a reason for hiding this comment

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

Not familiar with the new IA but do we not have these subareas anymore? It would still be helpful for domain-specific tutorials to be assigned to specific triagers/reviewers.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah I agree, I already started mapping these out in b328ad5 but I'm hesitant about it due to how much flux may happen here soon.

Copy link
Member

Choose a reason for hiding this comment

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

Could you keep the other ones as comments or empty arrays? I want us to eventually use the exactly same set of labels unless you plan to remove or add entire areas,

Copy link
Member Author

Choose a reason for hiding this comment

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

keep the [deleted] ones as comments or empty arrays

Good idea, I'll take a look at doing this

Copy link
Member Author

Choose a reason for hiding this comment

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

Done in ff4610e

Copy link
Member

Choose a reason for hiding this comment

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

On second thought, is there a reason why the labels can't describe the IA at the time being? Unless the migration will be done in a week, it would still be nice to have fine-grained triaging for all the different topics, especially for directing them to different maintainers. If we have to update this file again after the move anyway, it won't cause extra work.

- '\/docs\/Learn\/(Common|Front|Getting|Learning|Performance)'
Content:Learn:Accessibility:
- '\/docs\/Learn\/Accessibility'
Content:Learn:Client-side:
- '\/docs\/Learn\/Tools_and_testing\/(Client-side|Understanding_client-side)'
Content:Learn:Cross-Browser-Testing:
- '\/docs\/Learn\/Tools_and_testing\/Cross_browser_testing'
Content:Learn:CSS:
- '\/docs\/Learn\/CSS'
Content:Learn:Django:
- '\/docs\/Learn\/Server-side\/Django'
Content:Learn:Express:
- '\/docs\/Learn\/Server-side\/Express_Nodejs'
Content:Learn:Forms:
- '\/docs\/Learn\/Forms'
Content:Learn:GitHub:
- '\/docs\/Learn\/Tools_and_testing\/GitHub'
Content:Learn:HTML:
- '\/docs\/Learn\/HTML'
Content:Learn:JavaScript:
- '\/docs\/Learn\/JavaScript'
- '\/docs\/Learn_web_development\/'
Content:Manifest:
- '\/docs\/Web\/Manifest'
Content:MathML:
Expand Down
54 changes: 1 addition & 53 deletions .github/labeler.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,62 +57,10 @@ Content:JS:
Content:Learn:
- changed-files:
- all-globs-to-any-file:
- files/en-us/learn/**
- "!files/en-us/learn/accessibility/**"
- "!files/en-us/learn/tools_and_testing/client-side_javascript_frameworks/**"
- "!files/en-us/learn/tools_and_testing/understanding_client-side_tools/**"
- "!files/en-us/learn/tools_and_testing/cross_browser_testing/**"
- "!files/en-us/learn/css/**"
- "!files/en-us/learn/server-side/django/**"
- "!files/en-us/learn/server-side/express_nodejs/**"
- "!files/en-us/learn/forms/**"
- "!files/en-us/learn/tools_and_testing/github/**"
- "!files/en-us/learn/html/**"
- "!files/en-us/learn/javascript/**"
- files/en-us/learn_web_development/**
- changed-files:
- any-glob-to-any-file:
- files/en-us/web/tutorials/**
Content:Learn:Accessibility:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/accessibility/**
Content:Learn:Client-side:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/tools_and_testing/client-side_javascript_frameworks/**
- files/en-us/learn/tools_and_testing/understanding_client-side_tools/**
Content:Learn:Cross-Browser-Testing:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/tools_and_testing/cross_browser_testing/**
Content:Learn:CSS:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/css/**
Content:Learn:Django:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/server-side/django/**
Content:Learn:Express:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/server-side/express_nodejs/**
Content:Learn:Forms:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/forms/**
Content:Learn:GitHub:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/tools_and_testing/github/**
Content:Learn:HTML:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/html/**
Content:Learn:JavaScript:
- changed-files:
- any-glob-to-any-file:
- files/en-us/learn/javascript/**
Content:Manifest:
- changed-files:
- any-glob-to-any-file:
Expand Down
Loading