Skip to content

[SPARK-31330] Automatically label PRs based on the paths they touch #28114

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

Closed
nchammas wants to merge 10 commits into from
Closed

[SPARK-31330] Automatically label PRs based on the paths they touch #28114

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
bf064f1
add autolabeler rules
nchammas Apr 3, 2020
0d19a74
tweak rules
nchammas Apr 3, 2020
d6ecb1a
tweak r
nchammas Apr 3, 2020
20b220c
Refine the list and add some more fine-grained items
HyukjinKwon Apr 10, 2020
84d0c32
Move Scala style configuration to BUILD tag
HyukjinKwon Apr 10, 2020
aadea31
Fix another mistake
HyukjinKwon Apr 10, 2020
8228cf4
Address my own comments
HyukjinKwon Apr 10, 2020
7db3491
Add some more compoenents, address comments
HyukjinKwon Apr 11, 2020
30a09c5
Add more fine grained R files
HyukjinKwon Apr 11, 2020
2ddde69
Just add SQL for R components too
HyukjinKwon Apr 11, 2020
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
Prev Previous commit
Next Next commit
tweak r
  • Loading branch information
@nchammas
nchammas committed Apr 3, 2020
commit d6ecb1acb13f7e451c9dd934df60534cdae29269
3 changes: 2 additions & 1 deletion .github/autolabeler.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,4 +50,5 @@ java:
- "/common/"
- "java/"
R:
- "/R/"
- "r/"
- "R/"
Copy link
Member

@HyukjinKwon HyukjinKwon Apr 10, 2020

Choose a reason for hiding this comment

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

According to https://github.com/kaelzhang/node-ignore#optionsignorecase-since-400 which the plugin uses, seems it's case-insensitive by default but I just kept it.