chore: moved execa comment to check_dependencies from dependencies_va…

…lidator
aws-amplify · Aug 29, 2024 · 9a0f60b · 9a0f60b
1 parent 5d5439e
commit 9a0f60b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/scripts/check_dependencies.ts b/scripts/check_dependencies.ts
@@ -24,6 +24,9 @@ await new DependenciesValidator(
   [['aws-cdk', 'aws-cdk-lib']],
   [
     {
+      // @aws-amplify/plugin-types can depend on execa@^5.1.1 as a workaround for https://github.com/aws-amplify/amplify-backend/issues/962
+      // all other packages must depend on execa@^8.0.1
+      // this can be removed once execa is patched
       dependencyName: 'execa',
       globalDependencyVersion: '^8.0.1',
       exceptions: [

diff --git a/scripts/components/dependencies_validator.ts b/scripts/components/dependencies_validator.ts
@@ -256,9 +256,6 @@ export class DependenciesValidator {
         (x) => x.dependencyName === packageName
       );
     if (inconsistentDependency) {
-      // @aws-amplify/plugin-types can depend on execa@^5.1.1 as a workaround for https://github.com/aws-amplify/amplify-backend/issues/962
-      // all other packages must depend on execa@^8.0.1
-      // this can be removed once execa is patched
       return (declarations) => {
         const validationResult = declarations.every(
           ({ dependentPackageName, version }) =>