Skip to content

Avoid labeling 'context' PRs as 'text-input' - #157650

Merged
auto-submit[bot] merged 1 commit into
flutter:masterfrom
jmagman:text-context-labeler
Oct 28, 2024
Merged

Avoid labeling 'context' PRs as 'text-input'#157650
auto-submit[bot] merged 1 commit into
flutter:masterfrom
jmagman:text-context-labeler

Conversation

@jmagman

@jmagmanjmagman commented Oct 25, 2024

Copy link
Copy Markdown
Member

#157646 is being labeled by the PR labeler as "text-input" because it edited "context_runner.dart".

##[debug] "**/*text*" pattern matched packages/flutter_tools/lib/src/context_runner.dart

https://github.com/flutter/flutter/actions/runs/11526508378/job/32090756495?pr=157646#step:2:134

Exclude the common "context" word from this label rule.

See negation example at https://github.com/actions/labeler/blob/main/README.md#basic-examples.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@jmagmanjmagman self-assigned this Oct 25, 2024
@jmagmanjmagman added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Oct 25, 2024

@christopherfujinochristopherfujino left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jmagmanjmagman added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 28, 2024
@auto-submit
auto-submitBot merged commit 4a46649 into flutter:masterOct 28, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 28, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 28, 2024
auto-submitBot pushed a commit to flutter/packages that referenced this pull request Oct 28, 2024
flutter/flutter@5a11904...3fb95c7
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from bf5bfeebf89f to 23b2c6d3b3b1 (3 revisions) (flutter/flutter#157736)
2024-10-28 magder@google.com Avoid labeling 'context' PRs as 'text-input' (flutter/flutter#157650)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `restorable_route_future.0.dart` (flutter/flutter#157708)
2024-10-28 reidbaker@google.com Update .ci.yaml have android sdk be 35v1 (flutter/flutter#157621)
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 2522789c4199 to bf5bfeebf89f (1 revision) (flutter/flutter#157713)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `restoration_mixin.0.dart` (flutter/flutter#157709)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `navigator.restorable_push_replacement.0.dart` (flutter/flutter#157704)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `focus_node.unfocus.0.dart` (flutter/flutter#157673)
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 9226b089d634 to 2522789c4199 (2 revisions) (flutter/flutter#157710)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add tests for `navigator_state.restorable_push.0.dart` (flutter/flutter#157667)
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 10e50bd42c42 to 9226b089d634 (1 revision) (flutter/flutter#157702)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `navigator_state.restorable_push_replacement.0.dart` (flutter/flutter#157668)
2024-10-28 srawlins@google.com Remove unsafe_html lint rule (flutter/flutter#157531)
2024-10-27 engine-flutter-autoroll@skia.org Roll Flutter Engine from 3162aaad0086 to 10e50bd42c42 (1 revision) (flutter/flutter#157690)
2024-10-27 ybz975218925@gmail.com Fix Scaffold extend body (flutter/flutter#157441)
2024-10-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from c9b8ac96f6ce to 3162aaad0086 (3 revisions) (flutter/flutter#157681)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC stuartmorgan@google.com,tarrinneal@google.com on the revert to ensure that a human
is aware of the problem.
To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@jmagmanjmagman mentioned this pull request Oct 29, 2024
9 tasks
@jmagman
jmagman deleted the text-context-labeler branch October 29, 2024 17:19
@jmagmanjmagman added the revert Autorevert PR (with "Reason for revert:" comment) label Oct 29, 2024
@jmagman

jmagman commented Oct 29, 2024

Copy link
Copy Markdown
MemberAuthor

Reason for revert: this made the labeler go crazy and put it on everything that didn't contain "context".
#157724 (comment)

auto-submitBot pushed a commit that referenced this pull request Oct 29, 2024
@auto-submitauto-submitBot removed the revert Autorevert PR (with "Reason for revert:" comment) label Oct 29, 2024
auto-submitBot added a commit that referenced this pull request Oct 29, 2024
…7812)
Reverts: #157650
Initiated by: jmagman
Reason for reverting: this made the labeler go crazy and put it on everything that didn't contain "context".
Original PR Author: jmagman
Reviewed By: {christopherfujino}
This change reverts the following previous change:
#157646 is being labeled by the PR labeler as "text-input" because it edited "con**text**_runner.dart". ```
##[debug] "**/*text*" pattern matched packages/flutter_tools/lib/src/context_runner.dart
```
https://github.com/flutter/flutter/actions/runs/11526508378/job/32090756495?pr=157646#step:2:134
Exclude the common "context" word from this label rule.
See negation example at https://github.com/actions/labeler/blob/main/README.md#basic-examples.
auto-submitBot pushed a commit that referenced this pull request Oct 29, 2024
After #157650 all PRs are being labeled as text-input by the PR labeler. I think the config is missing an `all` based on the [negation example](https://github.com/actions/labeler/blob/main/README.md#basic-examples)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 12, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 7, 2025
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
flutter/flutter@5a11904...3fb95c7
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from bf5bfeebf89f to 23b2c6d3b3b1 (3 revisions) (flutter/flutter#157736)
2024-10-28 magder@google.com Avoid labeling 'context' PRs as 'text-input' (flutter/flutter#157650)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `restorable_route_future.0.dart` (flutter/flutter#157708)
2024-10-28 reidbaker@google.com Update .ci.yaml have android sdk be 35v1 (flutter/flutter#157621)
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 2522789c4199 to bf5bfeebf89f (1 revision) (flutter/flutter#157713)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `restoration_mixin.0.dart` (flutter/flutter#157709)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `navigator.restorable_push_replacement.0.dart` (flutter/flutter#157704)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `focus_node.unfocus.0.dart` (flutter/flutter#157673)
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 9226b089d634 to 2522789c4199 (2 revisions) (flutter/flutter#157710)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add tests for `navigator_state.restorable_push.0.dart` (flutter/flutter#157667)
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 10e50bd42c42 to 9226b089d634 (1 revision) (flutter/flutter#157702)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `navigator_state.restorable_push_replacement.0.dart` (flutter/flutter#157668)
2024-10-28 srawlins@google.com Remove unsafe_html lint rule (flutter/flutter#157531)
2024-10-27 engine-flutter-autoroll@skia.org Roll Flutter Engine from 3162aaad0086 to 10e50bd42c42 (1 revision) (flutter/flutter#157690)
2024-10-27 ybz975218925@gmail.com Fix Scaffold extend body (flutter/flutter#157441)
2024-10-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from c9b8ac96f6ce to 3162aaad0086 (3 revisions) (flutter/flutter#157681)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC stuartmorgan@google.com,tarrinneal@google.com on the revert to ensure that a human
is aware of the problem.
To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
flutter/flutter@5a11904...3fb95c7
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from bf5bfeebf89f to 23b2c6d3b3b1 (3 revisions) (flutter/flutter#157736)
2024-10-28 magder@google.com Avoid labeling 'context' PRs as 'text-input' (flutter/flutter#157650)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `restorable_route_future.0.dart` (flutter/flutter#157708)
2024-10-28 reidbaker@google.com Update .ci.yaml have android sdk be 35v1 (flutter/flutter#157621)
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 2522789c4199 to bf5bfeebf89f (1 revision) (flutter/flutter#157713)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `restoration_mixin.0.dart` (flutter/flutter#157709)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `navigator.restorable_push_replacement.0.dart` (flutter/flutter#157704)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `focus_node.unfocus.0.dart` (flutter/flutter#157673)
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 9226b089d634 to 2522789c4199 (2 revisions) (flutter/flutter#157710)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add tests for `navigator_state.restorable_push.0.dart` (flutter/flutter#157667)
2024-10-28 engine-flutter-autoroll@skia.org Roll Flutter Engine from 10e50bd42c42 to 9226b089d634 (1 revision) (flutter/flutter#157702)
2024-10-28 32538273+ValentinVignal@users.noreply.github.com Add test for `navigator_state.restorable_push_replacement.0.dart` (flutter/flutter#157668)
2024-10-28 srawlins@google.com Remove unsafe_html lint rule (flutter/flutter#157531)
2024-10-27 engine-flutter-autoroll@skia.org Roll Flutter Engine from 3162aaad0086 to 10e50bd42c42 (1 revision) (flutter/flutter#157690)
2024-10-27 ybz975218925@gmail.com Fix Scaffold extend body (flutter/flutter#157441)
2024-10-26 engine-flutter-autoroll@skia.org Roll Flutter Engine from c9b8ac96f6ce to 3162aaad0086 (3 revisions) (flutter/flutter#157681)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC stuartmorgan@google.com,tarrinneal@google.com on the revert to ensure that a human
is aware of the problem.
To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmitMerge PR when tree becomes green via auto submit Appc: contributor-productivityTeam-specific productivity, code health, technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jmagman@christopherfujino