refactor: update text on patches #445
Conversation
✅ Deploy Preview for vcluster-docs-site ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
joaniekube
force-pushed
the
joan_test-patch
branch
3 times, most recently
from
70bfada to
b538209
Compare
There was a problem hiding this comment.
@joaniefromtheblock I had to approve/trigger the vale workflow (it needs to be done for all external contributors). As you can see, there are some vale rules violations. Could you please fix those? Please refer to the CONTRIBUTING.md to learn more about styling guidelines and working with vale.
Tip
you can find the exact rules violations in the Files changed tab
change file location change file location change file location change file location change file location change file location change file location change file location change file location change file location change file location remove admonitions change file location change file location change file location update update link update link update link update link update link update link update link update link update link update first line updates for linter update text for clarity update text for consistency slight rewrite
joaniekube
force-pushed
the
joan_test-patch
branch
from
fb57a04 to
4c6139c
Compare
Co-authored-by: nebelsprachen <[email protected]>
|
@deniseschannon and @sowmyav27 Could you please take another look at this PR? Please let me know what additional changes may be needed or approve if ready. |
|
@Piotr1215 Could you please approve this PR ? |
There was a problem hiding this comment.
@joaniekube and I discussed offline on how the original code blocks were being templated but neither of us were aware of it and how we need to bring back the template using the props variables and how to generalize the explanations.
|
Hi @deniseschannon PTAL. I applied suggestions. Can this be approved? |
Content Description
contextvariablePreview Link
Internal Reference
N/A