Skip to content
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

Fix documentation for meta block: string replacement in key from - to _ #15940

Merged
merged 3 commits into from
Jan 30, 2023

Conversation

dttung2905
Copy link
Contributor

Hi team,

This PR will solve the issue #15359

Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
@jrasell jrasell self-assigned this Jan 30, 2023
@jrasell jrasell self-requested a review January 30, 2023 11:40
Copy link
Member

@jrasell jrasell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @dttung2905 and thanks for raising this PR. It is looking good, however, we don't require changelog entries for documentation updates. Could you remove this, I can then get this merged. Thanks.

Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
@dttung2905
Copy link
Contributor Author

Thank @jrasell for the super quick review. I have removed the changelog like you suggested. You also answered what I had in mind already when I searched for past PR by docs tag but found no related changelog too. 🙏

@jrasell jrasell added backport/website This will backport PR changes to `stable-website` && the latest release-branch backport/1.2.x backport to 1.1.x release line backport/1.3.x backport to 1.3.x release line backport/1.4.x backport to 1.4.x release line labels Jan 30, 2023
Copy link
Member

@jrasell jrasell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @dttung2905!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/website This will backport PR changes to `stable-website` && the latest release-branch backport/1.2.x backport to 1.1.x release line backport/1.3.x backport to 1.3.x release line backport/1.4.x backport to 1.4.x release line
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants