Skip to content

fix: amplify --version output warnings are interpreted as shell parameters#14797

Merged
iliapolo merged 1 commit intogen2-migrationfrom
iankhou-gen2-migration-version-parse
Apr 22, 2026
Merged

fix: amplify --version output warnings are interpreted as shell parameters#14797
iliapolo merged 1 commit intogen2-migrationfrom
iankhou-gen2-migration-version-parse

Conversation

@iankhou
Copy link
Copy Markdown
Contributor

@iankhou iankhou commented Apr 22, 2026

Description of changes

See https://tiny.amazon.com/63vy03uy/IsenLink.

We get the following error:

Unknown options: following,official,plugins,are,missing,or,inactive:,core:,core,\|,@aws-amplify/cli-internal-gen2-migration-experimental-alpha@14.3.0-dev.ae14ef198858c6fd50b6ed0196badda11a265735.0,14.3.0-dev.ae14ef198858c6fd50b6ed0196badda11a265735.0/amplify-pkg-win-x64.tgz

in the package binary upload step, where the version number is used as an input to a shell command. When --version returns extra output, like a warning, we get the error above.

This change removes output except for the last line, in uploadPkgCliCodeBuild.

Checklist

  • PR description included
  • yarn test passes
  • Tests are changed or added
  • Relevant documentation is changed or added (and PR referenced)
  • New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
  • Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@iankhou iankhou marked this pull request as ready for review April 22, 2026 21:21
@iankhou iankhou requested a review from a team as a code owner April 22, 2026 21:21
@iliapolo iliapolo merged commit 8eaf117 into gen2-migration Apr 22, 2026
4 checks passed
@iliapolo iliapolo deleted the iankhou-gen2-migration-version-parse branch April 22, 2026 23:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants