fix: Only run pub get in workspace root (#822)

<!--
  Thanks for contributing!

Provide a description of your changes below and a general summary in the
title

Please look at the following checklist to ensure that your PR can be
accepted quickly:
-->

## Description

We no longer need to run `pub get` in all packages, in one package, or
the workspace root is enough.

Fixes the issue mentioned here:
#816 (comment)

## Type of Change

<!--- Put an `x` in all the boxes that apply: -->

- [ ] ✨ `feat` -- New feature (non-breaking change which adds
functionality)
- [x] 🛠️ `fix` -- Bug fix (non-breaking change which fixes an issue)
- [ ] ❌ `!` -- Breaking change (fix or feature that would cause existing
functionality to change)
- [ ] 🧹 `refactor` -- Code refactor
- [ ] ✅ `ci` -- Build configuration change
- [ ] 📝 `docs` -- Documentation
- [ ] 🗑️ `chore` -- Chore

docs/commands/bootstrap.mdx

@@ -27,9 +27,9 @@ Bootstrapping has three primary functions:
 After the [Pub Workspaces feature](https://dart.dev/tools/pub/workspaces) was
 introduced in Dart 3.6.0, it is no longer strictly necessary to run `melos
 bootstrap`, since all the packages are already linked together. However, there
-are still some benefits to running `melos bootstrap`, because you need to run
-`pub get` in each package to initialize the workspace, and `melos bootstrap`
-will do that for you.
+are still some benefits to running `melos bootstrap`, if you for example want
+to set up shared dependencies or attaching your own setup scripts to the
+bootstrap hooks.
 
 ### Benefits
 

packages/melos/lib/src/commands/bootstrap.dart

@@ -60,7 +60,7 @@ mixin _BootstrapMixin on _CleanMixin {
           }
 
           logger.log(
-            'Running "$pubCommandForLogging" in workspace packages...',
+            'Running "$pubCommandForLogging" in workspace...',
           );
 
           await _runPubGetForWorkspace(
@@ -107,23 +107,6 @@ mixin _BootstrapMixin on _CleanMixin {
       runOffline: runOffline,
       enforceLockfile: enforceLockfile,
     );
-
-    final filteredPackages = workspace.filteredPackages.values;
-    await Stream.fromIterable(filteredPackages).parallel(
-      (package) async {
-        await runPubGetForPackage(
-          workspace,
-          package,
-          noExample: noExample,
-          runOffline: runOffline,
-          enforceLockfile: enforceLockfile,
-        );
-
-        _logPackagePubGetSuccess(package);
-      },
-      parallelism:
-          workspace.config.commands.bootstrap.runPubGetInParallel ? null : 1,
-    ).drain<void>();
   }
 
   @visibleForTesting
@@ -310,12 +293,6 @@ mixin _BootstrapMixin on _CleanMixin {
     return dependenciesToUpdate.length;
   }
 
-  void _logPackagePubGetSuccess(Package package) {
-    logger.child(packageNameStyle(package.name), prefix: '$checkLabel ').child(
-          packagePathStyle(printablePath(package.pathRelativeToWorkspace)),
-        );
-  }
-
   void _logBootstrapException(
     BootstrapException exception,
     MelosWorkspace workspace,

packages/melos/test/commands/bootstrap_test.dart

@@ -96,9 +96,7 @@ void main() {
 melos bootstrap
   └> ${workspaceDir.path}
 
-Running "${pubExecArgs.join(' ')} get" in workspace packages...
-  ✓ a
-    └> packages/a
+Running "${pubExecArgs.join(' ')} get" in workspace...
   > SUCCESS
 
 Generating IntelliJ IDE files...
@@ -280,7 +278,7 @@ Generating IntelliJ IDE files...
 melos bootstrap
   └> ${workspaceDir.path}
 
-Running "${pubExecArgs.join(' ')} get" in workspace packages...
+Running "${pubExecArgs.join(' ')} get" in workspace...
   - workspace
     └> .
 e-       └> Failed to run pub get.
@@ -331,7 +329,7 @@ e-Because a depends on package_that_does_not_exists any which doesn't exist (cou
 melos bootstrap
   └> ${workspaceDir.path}
 
-Running "${pubExecArgs.join(' ')} get --offline" in workspace packages...
+Running "${pubExecArgs.join(' ')} get --offline" in workspace...
   > SUCCESS
 
 Generating IntelliJ IDE files...
@@ -391,7 +389,7 @@ Generating IntelliJ IDE files...
 melos bootstrap
   └> ${workspaceDir.path}
 
-Running "${pubExecArgs.join(' ')} get --enforce-lockfile" in workspace packages...
+Running "${pubExecArgs.join(' ')} get --enforce-lockfile" in workspace...
   > SUCCESS
 
 Generating IntelliJ IDE files...
@@ -448,7 +446,7 @@ Generating IntelliJ IDE files...
 melos bootstrap
   └> ${workspaceDir.path}
 
-Running "${pubExecArgs.join(' ')} get" in workspace packages...
+Running "${pubExecArgs.join(' ')} get" in workspace...
   > SUCCESS
 
 Generating IntelliJ IDE files...
@@ -499,7 +497,7 @@ Generating IntelliJ IDE files...
 melos bootstrap
   └> ${workspaceDir.path}
 
-Running "${pubExecArgs.join(' ')} get" in workspace packages...
+Running "${pubExecArgs.join(' ')} get" in workspace...
   > SUCCESS
 
 Generating IntelliJ IDE files...
@@ -749,9 +747,7 @@ Generating IntelliJ IDE files...
 melos bootstrap
   └> ${workspaceDir.path}
 
-Running "dart pub get --offline" in workspace packages...
-  ✓ a
-    └> packages/a
+Running "dart pub get --offline" in workspace...
   > SUCCESS
 
 Generating IntelliJ IDE files...