Skip to content

Commit

Permalink
fix(actions): doc labeler needs all clause instead of default any (#5568
Browse files Browse the repository at this point in the history
)

Unspecified base match in labeler assumes 'any' for each match
clause. When specifying base-branch and --any-glob-to-any-file either
one of these cases would result in a successful match which would label
all PRs again main as documentation. We need to explicitly specify
'all:' in our labeler match config to ensure BOTH:

 * matching file paths related to documentation
         -AND-
 * targeting a merge against 'main' branch
  • Loading branch information
blackboxsw authored Jul 29, 2024
1 parent f9ab856 commit f8c1b51
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions .github/labeler.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
documentation:
- changed-files:
- any-glob-to-any-file:
- 'doc/*'
- 'cloudinit/config/schema/*'
- base-branch: 'main'
- all:
- changed-files:
- any-glob-to-any-file:
- 'doc/*'
- 'cloudinit/config/schema/*'
- base-branch: 'main'

0 comments on commit f8c1b51

Please sign in to comment.