-
Notifications
You must be signed in to change notification settings - Fork 816
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: container based deployments on native projects #6201
fix: container based deployments on native projects #6201
Conversation
c758cb4
to
68ec326
Compare
This pull request introduces 2 alerts when merging 15976be into af54abe - view on LGTM.com new alerts:
|
Codecov Report
@@ Coverage Diff @@
## master #6201 +/- ##
=======================================
Coverage 57.01% 57.01%
=======================================
Files 467 467
Lines 21472 21472
Branches 4267 4267
=======================================
Hits 12242 12242
Misses 8255 8255
Partials 975 975 Continue to review full report at Codecov.
|
be18b2f
to
b5bddc2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
packages/amplify-provider-awscloudformation/src/configuration-manager.ts
Outdated
Show resolved
Hide resolved
packages/amplify-provider-awscloudformation/src/configuration-manager.ts
Outdated
Show resolved
Hide resolved
packages/amplify-provider-awscloudformation/src/configuration-manager.ts
Outdated
Show resolved
Hide resolved
This pull request has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs. Looking for a help forum? We recommend joining the Amplify Community Discord server |
Issue #, if available:
Description of changes:
Native projects were missing to enable container based deployments
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.