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.
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
Clearer text about what happens with --disable-telemetry
+ enable-telemetry command
#125995
Clearer text about what happens with --disable-telemetry
+ enable-telemetry command
#125995
Conversation
final Usage legacyAnalytics = TestUsage(); | ||
setUp(() { | ||
legacyAnalytics.enabled = false; | ||
}); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wasn't being used anywhere in the tests, removing
packages/flutter_tools/lib/src/runner/flutter_command_runner.dart
Outdated
Show resolved
Hide resolved
throwToolExit( | ||
'Both enable and disable telemetry commands were detected ' | ||
'when only one can be supplied per invocation.', | ||
exitCode: 1); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is an exit code of 1
appropriate for this exit code? @gspencergoog
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I think so. Anything non-zero would work.
@gspencergoog friendly ping, does this look good to go? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…+ enable-telemetry command (flutter/flutter#125995)
flutter/flutter@43ac23b...4ed1c92 2023-05-08 engine-flutter-autoroll@skia.org Roll Flutter Engine from c42dd23e29f6 to a8e9ac63fd95 (1 revision) (flutter/flutter#126265) 2023-05-08 42216813+eliasyishak@users.noreply.github.com Clearer text about what happens with `--disable-telemetry` + enable-telemetry command (flutter/flutter#125995) 2023-05-08 engine-flutter-autoroll@skia.org Roll Flutter Engine from 23f730efbff7 to c42dd23e29f6 (13 revisions) (flutter/flutter#126262) 2023-05-08 goderbauer@google.com Fix leaks (flutter/flutter#126144) 2023-05-08 6655696+guidezpl@users.noreply.github.com Update packages (flutter/flutter#126140) 2023-05-06 pateltirth454@gmail.com [ExpansionPanelList] add materialGapSize property in ExpansionPanelList Widget (flutter/flutter#123971) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from eedcee66a710 to 23f730efbff7 (1 revision) (flutter/flutter#126199) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 393b5649e7b4 to eedcee66a710 (1 revision) (flutter/flutter#126198) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 876968833e46 to 393b5649e7b4 (1 revision) (flutter/flutter#126195) 2023-05-06 linxunfeng@yeah.net [ios] fix hold and drag spacebar does not move cursor when obscureTex� (flutter/flutter#122383) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from d9116d94a111 to 876968833e46 (1 revision) (flutter/flutter#126192) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 332f53219c4c to d9116d94a111 (1 revision) (flutter/flutter#126190) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from b7c79b41b9d4 to 332f53219c4c (3 revisions) (flutter/flutter#126188) 2023-05-06 github@alexv525.com � Add `rect` argument to `_ArcPaintPredicate` for tests (flutter/flutter#125953) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 758cbadfac1f to b7c79b41b9d4 (4 revisions) (flutter/flutter#126187) 2023-05-06 737941+loic-sharma@users.noreply.github.com Add Windows and Linux support to the 'layers' example (flutter/flutter#126105) 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 camillesimon@google.com,rmistry@google.com,stuartmorgan@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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
…r#3942) flutter/flutter@43ac23b...4ed1c92 2023-05-08 engine-flutter-autoroll@skia.org Roll Flutter Engine from c42dd23e29f6 to a8e9ac63fd95 (1 revision) (flutter/flutter#126265) 2023-05-08 42216813+eliasyishak@users.noreply.github.com Clearer text about what happens with `--disable-telemetry` + enable-telemetry command (flutter/flutter#125995) 2023-05-08 engine-flutter-autoroll@skia.org Roll Flutter Engine from 23f730efbff7 to c42dd23e29f6 (13 revisions) (flutter/flutter#126262) 2023-05-08 goderbauer@google.com Fix leaks (flutter/flutter#126144) 2023-05-08 6655696+guidezpl@users.noreply.github.com Update packages (flutter/flutter#126140) 2023-05-06 pateltirth454@gmail.com [ExpansionPanelList] add materialGapSize property in ExpansionPanelList Widget (flutter/flutter#123971) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from eedcee66a710 to 23f730efbff7 (1 revision) (flutter/flutter#126199) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 393b5649e7b4 to eedcee66a710 (1 revision) (flutter/flutter#126198) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 876968833e46 to 393b5649e7b4 (1 revision) (flutter/flutter#126195) 2023-05-06 linxunfeng@yeah.net [ios] fix hold and drag spacebar does not move cursor when obscureTexâ�¦ (flutter/flutter#122383) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from d9116d94a111 to 876968833e46 (1 revision) (flutter/flutter#126192) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 332f53219c4c to d9116d94a111 (1 revision) (flutter/flutter#126190) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from b7c79b41b9d4 to 332f53219c4c (3 revisions) (flutter/flutter#126188) 2023-05-06 github@alexv525.com â�¨ Add `rect` argument to `_ArcPaintPredicate` for tests (flutter/flutter#125953) 2023-05-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 758cbadfac1f to b7c79b41b9d4 (4 revisions) (flutter/flutter#126187) 2023-05-06 737941+loic-sharma@users.noreply.github.com Add Windows and Linux support to the 'layers' example (flutter/flutter#126105) 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 camillesimon@google.com,rmistry@google.com,stuartmorgan@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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
…+ enable-telemetry command (flutter/flutter#125995)
…+ enable-telemetry command (flutter/flutter#125995)
…+ enable-telemetry command (flutter/flutter#125995)
…+ enable-telemetry command (flutter/flutter#125995)
Fixes:
This PR is cleaning up the
--disable-telemetry
help message to make it clear that opting out will opt out of all telemetry collection for flutter and dart commands. It is also adding the opposite flag--enable-telemetry
which will enable telemetry collectionPre-launch Checklist
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.