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

Revert "fix(cli): better error message when trying to publish with CD configured in Amplify Console (#7874) #8106

Merged
merged 2 commits into from
Sep 7, 2021

Conversation

cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Sep 6, 2021

Description of changes

#7874 is causing customer issues in v5.4.0. Revert the change for now.

Issue #, if available

#8092 and a couple duplicates

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

This reverts commit f7e62df.

The commit being reverted builds on top of another commit that
is being reverted.
… configured in Amplify Console (#7874)"

This reverts commit 4f03859.

The commit being reverted causes a number of customer bug reports
in verion 5.4.0.
@cjihrig cjihrig requested a review from a team as a code owner September 6, 2021 14:57
@cjihrig cjihrig linked an issue Sep 6, 2021 that may be closed by this pull request
4 tasks
@codecov-commenter
Copy link

Codecov Report

Merging #8106 (abfc1ba) into master (b524870) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8106      +/-   ##
==========================================
- Coverage   54.43%   54.42%   -0.01%     
==========================================
  Files         664      663       -1     
  Lines       37838    37829       -9     
  Branches     7768     7767       -1     
==========================================
- Hits        20597    20589       -8     
+ Misses      16401    16400       -1     
  Partials      840      840              

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 b524870...abfc1ba. Read the comment docs.

@jhockett jhockett merged commit 61b7fc3 into aws-amplify:master Sep 7, 2021
@cjihrig cjihrig deleted the revert branch September 7, 2021 18:09
@github-actions
Copy link

github-actions bot commented Sep 9, 2021

👋 Hi, this pull request was referenced in the v5.5.0 release!

Check out the release notes here https://github.com/aws-amplify/amplify-cli/releases/tag/v5.5.0.

@github-actions github-actions bot added the referenced-in-release Issues referenced in a published release changelog label Sep 9, 2021
marcvberg pushed a commit to marcvberg/amplify-cli that referenced this pull request Oct 13, 2021
… configured in Amplify Console (aws-amplify#7874) (aws-amplify#8106)

* Revert "fix: handle case where there is no appId (aws-amplify#8049)"

This reverts commit f7e62df.

The commit being reverted builds on top of another commit that
is being reverted.

* Revert "fix(cli): better error message when trying to publish with CD configured in Amplify Console (aws-amplify#7874)"

This reverts commit 4f03859.

The commit being reverted causes a number of customer bug reports
in verion 5.4.0.

Co-authored-by: Colin Ihrig <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
referenced-in-release Issues referenced in a published release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Amplify Publish fails for manually deployed branch
4 participants