-
Notifications
You must be signed in to change notification settings - Fork 116
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
update composition environment docs to function-environment-configs #824
update composition environment docs to function-environment-configs #824
Conversation
✅ Deploy Preview for crossplane ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
4c6a437
to
25c0a00
Compare
Vale 😩🥸🤡🫠 |
ahh, putting aside vale issues, it looks like the build itself is failing. I'll take a pass through this PR to try to get everything ready 👍
|
Signed-off-by: Philippe Scorsolini <[email protected]>
Signed-off-by: Jared Watts <[email protected]>
25c0a00
to
ff712f0
Compare
Signed-off-by: Jared Watts <[email protected]>
Signed-off-by: Jared Watts <[email protected]>
Take this PR out of draft status now that all Vale errors and HTML link errors are fixed. Note the Vale check is still failing, but only with Warnings (0 Errors), which is acceptable. I'll focus on reviewing the content now. |
Signed-off-by: Jared Watts <[email protected]>
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.
I took a full pass through these updates and fixed everything I could find @phisco! This looks good to me to ship now, as long as the final checks/build pass ✅
This PR updates the docs to include information about how to use
EnvironmentConfigs
withfunction-environment-configs
. This will help Crossplane users migrate theirCompositions
due to the impact of maturingEnvironmentConfig
to Beta while removing native composition support for them in crossplane/crossplane#5938.