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

Simplify automated changelog #4525

Merged
merged 1 commit into from
Oct 16, 2023
Merged

Simplify automated changelog #4525

merged 1 commit into from
Oct 16, 2023

Conversation

lucacome
Copy link
Member

@lucacome lucacome commented Oct 16, 2023

Proposed changes

Removes unnecessary entries in the CHANGELOG. Our changelog is on https://github.com/nginxinc/kubernetes-ingress/releases and http://docs.nginx.com/nginx-ingress-controller/releases/ no need to add a new line in CHANGELOG.md on every release.

@lucacome lucacome self-assigned this Oct 16, 2023
@lucacome lucacome requested a review from a team as a code owner October 16, 2023 14:42
@github-actions github-actions bot added the documentation Pull requests/issues for documentation label Oct 16, 2023
## 1.11.0

An automatically generated list of changes can be found on GitHub at: [1.11.0 Release](https://github.com/nginxinc/kubernetes-ingress/releases/tag/v1.11.0)
Starting with version 1.11.0, an automatically generated list of changes can be found on the [GitHub Releases page](https://github.com/nginxinc/kubernetes-ingress/releases).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Starting with version 1.11.0, an automatically generated list of changes can be found on the [GitHub Releases page](https://github.com/nginxinc/kubernetes-ingress/releases).
Starting with version 1.11.0, an automatically generated list of changes can be found on the [GitHub Releases page](https://github.com/nginxinc/kubernetes-ingress/releases).
A curated list of changes can be found on the [Releases](http://docs.nginx.com/nginx-ingress-controller/releases/) page
on the NGINX Documentation website.

I think it would be useful to keep the link to releases page on our docs as well, since that also will direct the user to the latest release and all other releases.

@codecov
Copy link

codecov bot commented Oct 16, 2023

Codecov Report

Merging #4525 (1815095) into main (547e5bf) will decrease coverage by 0.02%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4525      +/-   ##
==========================================
- Coverage   52.00%   51.98%   -0.02%     
==========================================
  Files          59       59              
  Lines       16965    16965              
==========================================
- Hits         8822     8820       -2     
- Misses       7848     7850       +2     
  Partials      295      295              

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lucacome lucacome merged commit f6deac5 into main Oct 16, 2023
36 checks passed
@lucacome lucacome deleted the docs/changelog branch October 16, 2023 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Pull requests/issues for documentation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants