From 5c126e72415e009beec635326675b5e2859e99bc Mon Sep 17 00:00:00 2001 From: Stuart Morgan Date: Tue, 20 Jun 2023 14:37:01 -0400 Subject: [PATCH] [tools] Fix format instructions The instructions for `format` failures still referred to using `activate`, which is no longer how tools are invoked. Replace the specific instructions with a link to the relevant README section to make them evergreen. --- script/tool/lib/src/format_command.dart | 6 +++--- script/tool/test/format_command_test.dart | 3 +++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/script/tool/lib/src/format_command.dart b/script/tool/lib/src/format_command.dart index 36fd063b2b49..30a5565f2322 100644 --- a/script/tool/lib/src/format_command.dart +++ b/script/tool/lib/src/format_command.dart @@ -104,9 +104,9 @@ class FormatCommand extends PackageCommand { print('These files are not formatted correctly (see diff below):'); LineSplitter.split(stdout).map((String line) => ' $line').forEach(print); - print('\nTo fix run "dart pub global activate flutter_plugin_tools && ' - 'dart pub global run flutter_plugin_tools format" or copy-paste ' - 'this command into your terminal:'); + print('\nTo fix run the repository tooling `format` command: ' + 'https://github.com/flutter/packages/blob/main/script/tool/README.md#format-code\n' + 'or copy-paste this command into your terminal:'); final io.ProcessResult diff = await processRunner.run( 'git', diff --git a/script/tool/test/format_command_test.dart b/script/tool/test/format_command_test.dart index 546be9289fb0..86c3ffb0cca8 100644 --- a/script/tool/test/format_command_test.dart +++ b/script/tool/test/format_command_test.dart @@ -512,6 +512,9 @@ void main() { containsAllInOrder([ contains('These files are not formatted correctly'), contains(changedFilePath), + // Ensure the error message links to instructions. + contains( + 'https://github.com/flutter/packages/blob/main/script/tool/README.md#format-code'), contains('patch -p1 <