feat(breaking): standardize ims credential env vars, make them predictable and easy to add to actions #892
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.
Description
Breaking change
Proposing to make it so ims credential env vars are easy to pass to actions.
Today, when we
aio app useor app init, we get something like this in .env:This PR proposes standardizing these properties to something easier to predict, the solution is to prefix each field in the credential with
IMS_:And then they can be passed to Runtime actions:
For projects with multiple credentials, the env vars are postfixed with _{index}:
Related Issue
N/A
Motivation and Context
Make it easier to use IMS credential env vars
How Has This Been Tested?
npm run testaio app useandaio app initScreenshots (if appropriate):
Types of changes
Checklist: