docs: add link to a proper aws guide for configuring credentials#4091
Merged
mergify[bot] merged 5 commits intoaws:mainlinefrom Jan 18, 2023
Merged
docs: add link to a proper aws guide for configuring credentials#4091mergify[bot] merged 5 commits intoaws:mainlinefrom
mergify[bot] merged 5 commits intoaws:mainlinefrom
Conversation
Contributor
|
could you add #4089 (comment) and #4089 (comment)? i forgot to add the label to stop them from merging in that PR 🥺 |
Codecov Report
@@ Coverage Diff @@
## mainline #4091 +/- ##
=========================================
Coverage 69.17% 69.17%
=========================================
Files 248 248
Lines 35609 35609
Branches 264 264
=========================================
Hits 24634 24634
Misses 9781 9781
Partials 1194 1194
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Add a link on top of the credentials docs to a proper aws guide Signed-off-by: Sergey Generalov <sergey@genbit.ru>
Add "your" to indicate that error is on user side to fix Signed-off-by: Sergey Generalov <sergey@genbit.ru>
genbit
force-pushed
the
docs-credentials
branch
from
f8a83e1 to
ab90953
Compare
Contributor
Author
|
@dannyrandall done |
huanjani
reviewed
Oct 18, 2022
| This section explains our recommendations around credentials to provide the best experience with the AWS Copilot CLI. | ||
| AWS Copilot uses AWS credentials to access AWS API, store and look up an [application's metadata](concepts/applications.en.md), and deploy and operate application's workloads. | ||
|
|
||
| You can learn more on how to configure AWS credentials at [Quick configuration with `aws configure`](https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-quickstart.html#cli-configure-quickstart-config). |
Contributor
There was a problem hiding this comment.
This is super helpful for first-time users. I remember being confused by this when I first started! Thank you.
huanjani
reviewed
Oct 18, 2022
huanjani
added
do-not-merge
iamhopaul123
added
the
do-not-merge
Lou1415926
removed
the
do-not-merge
Lou1415926
added
do-not-merge
Co-authored-by: Janice Huang <60631893+huanjani@users.noreply.github.com>
Co-authored-by: Danny Randall <10566468+dannyrandall@users.noreply.github.com>
Lou1415926
approved these changes
Jan 18, 2023
|
🍕 Here are the new binary sizes!
|
huanjani
approved these changes
Jan 18, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add a link on top of the credentials docs to a proper aws guide
Signed-off-by: Sergey Generalov sergey@genbit.ru
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the Apache 2.0 License.