-
Notifications
You must be signed in to change notification settings - Fork 697
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
changelogs are docs #10354
changelogs are docs #10354
Conversation
701f670
to
5f9b068
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent
@ffaf1 did you hit comment instead of approve by mistake? |
I think comment is the default
…On Sat, Sep 14, 2024, 16:40 Artem Pelenitsyn ***@***.***> wrote:
@ffaf1 <https://github.com/ffaf1> did you hit comment instead of approve
by mistake?
—
Reply to this email directly, view it on GitHub
<#10354 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAPRFIERJVBSIM34457MVB3ZWSNLBAVCNFSM6AAAAABOGN6YIGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJRGE2DKMZRGE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
N.B.: I took the liberty of adding "needs-backport 3.14", but as it's now a bit too late for it to be relevant to the 3.14.0.0 release it should be backported afterward. 3.12 can be backported as soon as it's merged to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent
(this time selecting “Approve” first)
@mergify backport 3.12 |
✅ Backports have been created
|
Looks like I forgot some jobs that need to be updated; will fix shortly. |
5f9b068
to
3546323
Compare
We don't mark changelogs as documentation, so CI unnecessarily does full checks when we add changelogs. Correct this. NOTE: we only accept changelog files from top-level subdirectories. There are changelog files in various tests that must be considered to be "code".
3546323
to
871c0d4
Compare
changelogs are docs (backport #10354)
@mergify backport 3.14 |
✅ Backports have been created
|
We don't mark changelogs as documentation, so CI unnecessarily does full checks when we add changelogs. Correct this.
NOTE: we only accept changelog files from top-level subdirectories. There are changelog files in package tests that must be considered "code" from the standpoint of CI.
Template B: This PR does not modify behaviour or interface
E.g. the PR only touches documentation or tests, does refactorings, etc.
Include the following checklist in your PR: