--[no-]analytics is deprecated. Use --suppress-analytics to disable analytics for one run instead.

[dcharkes]

@jonasfj @sigurdm Do I need to check what version of Dart we're running before deciding which of the two flags to use? Or did --suppress-analytics get added a while back already?

[sigurdm]

--suppress-analytics was added to the sdk in 68353d76a495b10eee0c6fc8c05e8a836a8577b7

That was for v. 3.0.0 I believe.

> git co 68353d76a495b10eee0c6fc8c05e8a836a8577b7
> $ tail tools/VERSION 
#
#  * Making cherry-picks to stable channel
#     - increase PATCH by 1
#
CHANNEL be
MAJOR 3
MINOR 0
PATCH 0
PRERELEASE 0
PRERELEASE_PATCH 0

[sigurdm]

When are you using the flag?

I believe you can also suppress analytics by setting some env variable (Maybe CI or BOT) but that might be a misuse...

[dcharkes]

When are you using the flag?

native/pkgs/native_assets_builder/test/build_runner/helpers.dart

Lines 16 to 27 in fd21f5b

	Future<void> runPubGet({
	required Uri workingDirectory,
	required Logger logger,
	}) async {
	final result = await runProcess(
	executable: Uri.file(Platform.resolvedExecutable),
	arguments: ['pub', '--no-analytics', 'get'],
	workingDirectory: workingDirectory,
	logger: logger,
	);
	expect(result.exitCode, 0);
	}

It's probably fine to bump it to --suppress-analytics, we already require Dart SDK 3.0 in the pubspec.