Skip to content

docs: add helm-version and yaml-version shortcode#3766

Merged
zirain merged 5 commits intoenvoyproxy:mainfrom
zirain:shortcode/helm-version
Jul 24, 2024
Merged

docs: add helm-version and yaml-version shortcode#3766
zirain merged 5 commits intoenvoyproxy:mainfrom
zirain:shortcode/helm-version

Conversation

@zirain
Copy link
Copy Markdown
Member

@zirain zirain commented Jul 6, 2024

Thi will make it simpler to manage the version of helm chart and yaml file in doc site.

@zirain zirain requested a review from a team as a code owner July 6, 2024 11:11
@zirain zirain force-pushed the shortcode/helm-version branch from be93ea4 to 52ac84e Compare July 6, 2024 11:14
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.38%. Comparing base (9852333) to head (c1dace9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3766      +/-   ##
==========================================
- Coverage   67.39%   67.38%   -0.02%     
==========================================
  Files         183      183              
  Lines       22435    22435              
==========================================
- Hits        15121    15117       -4     
- Misses       6226     6229       +3     
- Partials     1088     1089       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@zirain
Copy link
Copy Markdown
Member Author

zirain commented Jul 6, 2024

/retest

@zirain zirain force-pushed the shortcode/helm-version branch 2 times, most recently from 7ea97d7 to 9b7f420 Compare July 16, 2024 12:45
@zhaohuabing
Copy link
Copy Markdown
Member

@zirain Why do we need this?

@zirain
Copy link
Copy Markdown
Member Author

zirain commented Jul 22, 2024

@zirain Why do we need this?

This will simply the work when cutting a new doc version.
The old way is broken for a long time.

shawnh2
shawnh2 previously approved these changes Jul 23, 2024
Copy link
Copy Markdown
Contributor

@shawnh2 shawnh2 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 for optimizing this!

@zirain zirain force-pushed the shortcode/helm-version branch 2 times, most recently from a844c98 to 148f54e Compare July 23, 2024 08:30
Signed-off-by: zirain <zirain2009@gmail.com>
@zirain zirain force-pushed the shortcode/helm-version branch from 148f54e to 1216b1c Compare July 23, 2024 08:31
zirain added 2 commits July 23, 2024 16:58
Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: zirain <zirain2009@gmail.com>
@zirain zirain changed the title docs: add helm-version shortcode docs: add helm-version and yaml-version shortcode Jul 23, 2024
@zirain zirain force-pushed the shortcode/helm-version branch from d05ab85 to d775201 Compare July 23, 2024 09:08
Signed-off-by: zirain <zirain2009@gmail.com>
@zirain zirain force-pushed the shortcode/helm-version branch from d775201 to c1dace9 Compare July 23, 2024 09:09
@zirain zirain requested review from shawnh2 and zhaohuabing July 23, 2024 10:01
Copy link
Copy Markdown
Member

@zhaohuabing zhaohuabing 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!

2. Create a release announcement. Refer to [PR #635] as an example release announcement.
3. Include the release in the compatibility matrix. Refer to [PR #1002] as an example.
4. Generate the versioned release docs:
1. Create a release announcement. Refer to [PR #635] as an example release announcement.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: those numbers are not correct

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@zirain
Copy link
Copy Markdown
Member Author

zirain commented Jul 24, 2024

wait for #3910

@zirain zirain requested a review from shawnh2 July 24, 2024 02:11
@zirain
Copy link
Copy Markdown
Member Author

zirain commented Jul 24, 2024

/retest

@zirain zirain merged commit c78f427 into envoyproxy:main Jul 24, 2024
@zirain zirain deleted the shortcode/helm-version branch July 24, 2024 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants