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

[25.0 backport] Require changelog in 25.0 #5278

Merged
merged 3 commits into from
Jul 22, 2024

Conversation

austinvazquez
Copy link
Contributor

- What I did
Backports #4981 to 25.0

- How I did it

git cherry-pick -xsS 745704d7b4ec55c93b84db5372e0369850434207
git cherry-pick -xsS f92fcdef1b5f801c2fe359eee1833473caeabb5a
git cherry-pick -xsS c3243a8cc3da6bafc9e363d3d09550e558cd529f

- How to verify it

- Description for the changelog

Require impactful changes to include a changelog entry and area label

- A picture of a cute animal (not mandatory but encouraged)

Any PR that is labeled with any `impact/*` label should have a
description for the changelog and an `area/*` label.

(copied from moby/moby 1d473549e865ef6b90ee936c280f4bda677de39b)

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
(cherry picked from commit 745704d)
Signed-off-by: Austin Vazquez <macedonv@amazon.com>
(copied from moby/moby 61269e718fbdbbad397b0089105ec910fc0e62ca)

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
(cherry picked from commit f92fcde)
Signed-off-by: Austin Vazquez <macedonv@amazon.com>
This will make Github render the log line as an error.

(copied from moby/moby fb92caf2aa6cf3664e11dc06ee10d114af300826)

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
(cherry picked from commit c3243a8)
Signed-off-by: Austin Vazquez <macedonv@amazon.com>
Copy link
Member

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

@codecov-commenter
Copy link

codecov-commenter commented Jul 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.36%. Comparing base (cdd733f) to head (26850c6).
Report is 10 commits behind head on 25.0.

Additional details and impacted files
@@            Coverage Diff             @@
##             25.0    #5278      +/-   ##
==========================================
+ Coverage   61.35%   61.36%   +0.01%     
==========================================
  Files         287      290       +3     
  Lines       20088    20098      +10     
==========================================
+ Hits        12324    12334      +10     
- Misses       6870     6871       +1     
+ Partials      894      893       -1     

@thaJeztah thaJeztah merged commit 4c0e910 into docker:25.0 Jul 22, 2024
95 checks passed
@austinvazquez austinvazquez deleted the require-changelog-in-25.0 branch July 22, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants