E549 Simplify release script by lucacome · Pull Request #4425 · nginx/kubernetes-ingress · GitHub
[go: up one dir, main page]

Skip to content

Conversation

lucacome
Copy link

Proposed changes

There's a single release script that can now be used for any kind of release.

@lucacome lucacome self-assigned this Sep 26, 2023
@lucacome lucacome requested review from a team as code owners September 26, 2023 01:39
@github-actions github-actions bot added chore Pull requests for routine tasks documentation Pull requests/issues for documentation labels Sep 26, 2023
Comment on lines +27 to +31
Copy link
Author

Choose a reason for hiding this comment

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

I don't think adding this to every minor release adds much value. Should we just add a link to https://docs.nginx.com/nginx-ingress-controller/technical-specifications/#supported-kubernetes-versions at the top of the page? @brianehlert @ADubhlaoich @Jcahilltorre

@codecov
Copy link
codecov bot commented Sep 26, 2023

Codecov Report

Merging #4425 (a223cbb) into main (9dcd4cc) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4425      +/-   ##
==========================================
- Coverage   52.12%   52.08%   -0.04%     
==========================================
  Files          59       59              
  Lines       16956    16956              
==========================================
- Hits         8838     8832       -6     
- Misses       7821     7825       +4     
- Partials      297      299       +2     

see 2 files with indirect coverage changes

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

@lucacome lucacome force-pushed the chore/release-script branch from 53325e3 to a223cbb Compare September 26, 2023 22:42
@lucacome lucacome merged commit ecc04a1 into main Sep 27, 2023
@lucacome lucacome deleted the chore/release-script branch September 27, 2023 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull requests for routine tasks documentation Pull requests/issues for documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0