Unverified Commit a82ae232 authored by Drew Roen's avatar Drew Roen Committed by GitHub

Update labeler version, fix yaml to work with v5 (#139564)

This fixes https://github.com/flutter/flutter/issues/139511

* A bug has been fixed with `sync-labels`, meaning this needs to be a boolean now. Setting to `true` to match v4 logic.
* in `labeler.yml`, all labels must be a list of globs, so updated them all to be a list of `any`.
* Update the version to v5

This is a little annoying to test, since the way github actions works means that changes to workflows aren't run until after they are merged. A workaround is I forked these labeler changes to a new branch in my own repo and created a PR to merge to that branch, seen here: https://github.com/drewroengoogle/flutter/actions/runs/7102118110/job/19331743809?pr=2. Note the step `Run actions/labeler@8558fd74291d67161a8a78ce36a881fa63b766a9` which ensures we are running on the latest labeler version in that check.
parent ea97fc48
...@@ -4,65 +4,80 @@ ...@@ -4,65 +4,80 @@
# See https://github.com/actions/labeler/blob/main/README.md for docs. # See https://github.com/actions/labeler/blob/main/README.md for docs.
'a: accessibility': 'a: accessibility':
- any:
- '**/accessibility/*' - '**/accessibility/*'
- '**/*accessibility*' - '**/*accessibility*'
- '**/semantics/*' - '**/semantics/*'
- '**/*semantics*' - '**/*semantics*'
'a: animation': 'a: animation':
- any:
- '**/animation/*' - '**/animation/*'
- '**/*animation*' - '**/*animation*'
'a: desktop': 'a: desktop':
- any:
- '**/linux/**/*' - '**/linux/**/*'
- '**/macos/**/*' - '**/macos/**/*'
- '**/windows/**/*' - '**/windows/**/*'
'a: internationalization': 'a: internationalization':
- any:
- packages/flutter_localizations/**/* - packages/flutter_localizations/**/*
'a: tests': 'a: tests':
- any:
- packages/flutter_driver/**/* - packages/flutter_driver/**/*
- packages/flutter_goldens/**/* - packages/flutter_goldens/**/*
- packages/flutter_goldens_client/**/* - packages/flutter_goldens_client/**/*
- packages/flutter_test/**/* - packages/flutter_test/**/*
'a: text input': 'a: text input':
- any:
- '**/text/*' - '**/text/*'
- '**/*text*' - '**/*text*'
'd: api docs': 'd: api docs':
- any:
- examples/api/**/* - examples/api/**/*
'd: examples': 'd: examples':
- any:
- examples/**/* - examples/**/*
engine: engine:
- any:
- bin/internal/engine.version - bin/internal/engine.version
'f: cupertino': 'f: cupertino':
- any:
- '**/cupertino/*' - '**/cupertino/*'
- '**/*cupertino*' - '**/*cupertino*'
'f: focus': 'f: focus':
- any:
- '**/focus/*' - '**/focus/*'
- '**/*focus*' - '**/*focus*'
'f: gestures': 'f: gestures':
- any:
- '**/gestures/*' - '**/gestures/*'
- '**/*gestures*' - '**/*gestures*'
'f: material design': 'f: material design':
- any:
- '**/material/*' - '**/material/*'
- '**/*material*' - '**/*material*'
'f: routes': 'f: routes':
- any:
- '**/navigator/*' - '**/navigator/*'
- '**/*navigator*' - '**/*navigator*'
- '**/route/*' - '**/route/*'
- '**/*route*' - '**/*route*'
'f: scrolling': 'f: scrolling':
- any:
- '**/*scroll*' - '**/*scroll*'
- '**/scroll/*' - '**/scroll/*'
- '**/*sliver*' - '**/*sliver*'
...@@ -71,6 +86,7 @@ engine: ...@@ -71,6 +86,7 @@ engine:
- '**/viewport/*' - '**/viewport/*'
framework: framework:
- any:
- packages/flutter/**/* - packages/flutter/**/*
- packages/flutter_driver/**/* - packages/flutter_driver/**/*
- packages/flutter_goldens/**/* - packages/flutter_goldens/**/*
...@@ -80,19 +96,24 @@ framework: ...@@ -80,19 +96,24 @@ framework:
- examples/api/**/* - examples/api/**/*
'f: integration_test': 'f: integration_test':
- any:
- packages/integration_test/**/* - packages/integration_test/**/*
platform-ios: platform-ios:
- any:
- packages/flutter_tools/lib/src/ios/**/* - packages/flutter_tools/lib/src/ios/**/*
'customer: gallery': 'customer: gallery':
- any:
- examples/flutter_gallery/**/* - examples/flutter_gallery/**/*
'c: tech-debt': 'c: tech-debt':
- any:
- '**/fix_data.yaml' - '**/fix_data.yaml'
- '**/*.expect' - '**/*.expect'
- '**/*test_fixes*' - '**/*test_fixes*'
tool: tool:
- any:
- packages/flutter_tools/**/* - packages/flutter_tools/**/*
- packages/fuchsia_remote_debug_protocol/**/* - packages/fuchsia_remote_debug_protocol/**/*
...@@ -17,6 +17,6 @@ jobs: ...@@ -17,6 +17,6 @@ jobs:
runs-on: ubuntu-latest runs-on: ubuntu-latest
steps: steps:
# Source available at https://github.com/actions/labeler/blob/main/README.md # Source available at https://github.com/actions/labeler/blob/main/README.md
- uses: actions/labeler@ac9175f8a1f3625fd0d4fb234536d26811351594 - uses: actions/labeler@8558fd74291d67161a8a78ce36a881fa63b766a9
with: with:
sync-labels: '' sync-labels: true
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment