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

chore: update release template #6883

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

mlavacca
Copy link
Member

What this PR does / why we need it:

Make the release instructions a bit clearer.

Which issue this PR fixes:

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@mlavacca mlavacca requested a review from a team as a code owner December 20, 2024 09:56
Copy link

codecov bot commented Dec 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.4%. Comparing base (b14df00) to head (1784b39).
Report is 8 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff            @@
##            main   #6883      +/-   ##
========================================
+ Coverage   59.0%   76.4%   +17.3%     
========================================
  Files        210     212       +2     
  Lines      25241   25246       +5     
========================================
+ Hits       14917   19312    +4395     
+ Misses      9030    4913    -4117     
+ Partials    1294    1021     -273     

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

- label: Ensure a draft PR is created in [docs.konghq.com](https://github.com/Kong/docs.konghq.com/pulls) repository.
- label: If you are adding a new CRD, add a new description file under `app/_includes/md/kic/crd-ref/`. This is a brief description injected into the CRD reference page.
- label: Update articles in the new version as needed.
- label: Update `reference/version-compatibility.md` to include the new versions (make sure you capture any new Kubernetes/Istio versions that have been tested)
- label: Copy `app/_data/docs_nav_kic_OLDVERSION.yml` to `app/_data/docs_nav_kic_NEWVERSION.yml` and update the `release` field to `NEWVERSION`. Add entries for any new articles.
- label: Make sure that `app/_data/docs_nav_kic_NEWVERSION.yml` links to the latest generated `custom-resources-X.X.X.md`.
- label: Make sure that `app/_data/docs_nav_kic_NEWVERSION.yml` custom resources link to the latest generated `custom-resources-X.X.X.md`.
- label: Make sure that `app/_data/docs_nav_kic_NEWVERSION.yml` cli arguments link to the latest generated `cli-arguments-X.X.X.md`.
Copy link
Contributor

Choose a reason for hiding this comment

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

You'll need to copy the files as well.

Copy link
Member Author

Choose a reason for hiding this comment

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

In Kong/docs.konghq.com#8305 we needed to manually copy the files because we were adding the nav for 3.5 upfront (as unreleased). I don't think this is something good to do, as the cli args and the custom-resources files were automatically generated by the bot in Kong/docs.konghq.com#8298.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes but you need some to have an unreleased version. Referencing the old ones risk forgetting them so I'm not sure what's best here. TBH I think whatever we end up doing in: Kong/gateway-operator#35 would make this a lot simpler

Copy link
Member Author

Choose a reason for hiding this comment

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

I've addressed it in 1784b39. PTAL

@mlavacca mlavacca requested a review from lahabana December 20, 2024 13:57
Signed-off-by: Mattia Lavacca <[email protected]>
Signed-off-by: Mattia Lavacca <[email protected]>
@mlavacca mlavacca force-pushed the update-release-template branch from 583d8ee to 1784b39 Compare December 23, 2024 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants