Fix wording in immutable fields note in managed resources - #1152
Conversation
✅ Deploy Preview for crossplane ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Signed-off-by: Marcelo Villa <36754005+marcelovilla@users.noreply.github.com>
dd0cf66 to
f71bce5
Compare
jbw976
left a comment
There was a problem hiding this comment.
nice catch, thank you for finding this! can you also make this same change in content/master? that's the version that will be the next release (e.g. v2.5), so it should be fixed there too.
otherwise, this edit looks good to me! (and it's a refreshingly human feeling contribution 😉)
Signed-off-by: Marcelo Villa <36754005+marcelovilla@users.noreply.github.com>
|
Thanks @jbw976, appreciate the quick review! Just pushed the same change to |
jbw976
left a comment
There was a problem hiding this comment.
awesome, thanks for making the update to master too! this looks good to me, just waiting for the queued CI to run before I merge 🙏
|
30 mins later and CI hasn't started, so I just ran Vale locally on the latest commit in this PR 610b28d and it succeeded ✅ Let's merge 😂 |

I was going over the docs at https://docs.crossplane.io/latest/managed-resources/managed-resources/ and noticed the note Immutable fields section reads weirdly. This is a tiny PR fixing that wording.