Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(cli): add noUpdateBackend cli argument for pull and add headless pull sample #8804

Conversation

awsaxeman
Copy link

Description of changes

Adds an option to the cli when using the pull option that lets you specify not to update the
backend (--noUpdateBackend) to be able to generate aws-exports.js. This is an alternative to --yes option which defaults
to modifying the backend and is good for frontend custom CICD deployments.

Issue #, if available

re #7468

Description of how you validated changes

Tested with pull script and was able to pull down an existing amplify app to an empty project directory.

Checklist

  • PR description included
  • yarn test passes

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

…pull sample

Adds an option to the cli when using the pull option that lets you specify not to update the
backend to be able to generate aws-exports.js. This is an alternative to yes option which defaults
to modifying the backend and is good for frontend custom CICD deployments.

re aws-amplify#7468
@awsaxeman awsaxeman requested a review from a team as a code owner November 11, 2021 17:15
@codecov-commenter
Copy link

Codecov Report

Merging #8804 (52448f7) into master (59a7325) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8804   +/-   ##
=======================================
  Coverage   57.20%   57.20%           
=======================================
  Files         757      757           
  Lines       42243    42243           
  Branches     8659     8659           
=======================================
  Hits        24166    24166           
  Misses      17263    17263           
  Partials      814      814           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 59a7325...52448f7. Read the comment docs.

@aws-eddy
Copy link
Contributor

Hey @awsaxeman, would you be interested in continuing to work on this PR? If so, please rebase this onto dev by running git rebase dev. We will help you with getting this PR to the finish line 🏁

@aws-eddy
Copy link
Contributor

We haven't heard back from you so we are going to close this pull request. If you are interested in pursuing it in the future, please reopen it.

@mtourj
Copy link

mtourj commented Nov 28, 2023

Hello @evcodes I've re-submitted the changes in this PR here: #13434
Our workflow would benefit from this feature 🙏 Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
first-time-contributor The contribution is the first for this user in the repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants