Skip to content

Conversation

@DaughterOfMars
Copy link
Contributor

Description of change

Adds a job which ensures that PR release notes are correct. It also now enforces that release notes have no empty check boxes, so in the case where there are no notes, the section should be deleted.

@vercel
Copy link

vercel bot commented Apr 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

6 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
apps-backend ⬜️ Ignored Preview Aug 13, 2025 7:11am
apps-ui-kit ⬜️ Ignored Preview Comment Aug 13, 2025 7:11am
iota-evm-bridge ⬜️ Ignored Preview Aug 13, 2025 7:11am
iota-multisig-toolkit ⬜️ Ignored Preview Aug 13, 2025 7:11am
rebased-explorer ⬜️ Ignored Preview Aug 13, 2025 7:11am
wallet-dashboard ⬜️ Ignored Preview Aug 13, 2025 7:11am

@github-actions github-actions bot added the ci Issues related to our CI pipeline label Apr 30, 2025
@iota-ci iota-ci added dev-tools Issues related to the Developer Tools Team sc-platform Issues related to the Smart Contract Platform group. labels Apr 30, 2025
@thibault-martinez
Copy link
Member

It also now enforces that release notes have no empty check boxes, so in the case where there are no notes, the section should be deleted.

I am not too sure about this, why? Seems annoying.

@thibault-martinez
Copy link
Member

It also now enforces that release notes have no empty check boxes, so in the case where there are no notes, the section should be deleted.

I am not too sure about this, why? Seems annoying.

A reminder for people that may have forgot to fill it?

)

if not any_checked and len(notes) > 0:
issues.append(f" - No checked items in release notes");
Copy link
Member

Choose a reason for hiding this comment

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

Does this mean it will fail if we left the templates notes empty? IMO we shouldn't check that, sounds very annoying.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You should remove it if you don't have any release notes

Copy link
Member

Choose a reason for hiding this comment

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

I disagree, but I approved anyway. We'll revert when/if too many people complain.

@thibault-martinez thibault-martinez merged commit 308cc9d into develop Aug 13, 2025
39 checks passed
@thibault-martinez thibault-martinez deleted the dev-tools/release-notes-ci branch August 13, 2025 08:15
roman1e2f5p8s pushed a commit that referenced this pull request Aug 28, 2025
# Description of change

Adds a job which ensures that PR release notes are correct. It also now
enforces that release notes have no empty check boxes, so in the case
where there are no notes, the section should be deleted.

---------

Co-authored-by: Thibault Martinez <thibault@iota.org>
roman1e2f5p8s pushed a commit that referenced this pull request Aug 28, 2025
# Description of change

Adds a job which ensures that PR release notes are correct. It also now
enforces that release notes have no empty check boxes, so in the case
where there are no notes, the section should be deleted.

---------

Co-authored-by: Thibault Martinez <thibault@iota.org>
roman1e2f5p8s pushed a commit that referenced this pull request Aug 28, 2025
# Description of change

Adds a job which ensures that PR release notes are correct. It also now
enforces that release notes have no empty check boxes, so in the case
where there are no notes, the section should be deleted.

---------

Co-authored-by: Thibault Martinez <thibault@iota.org>
Signed-off-by: Roman Overko <roman.overko@iota.org>
roman1e2f5p8s pushed a commit that referenced this pull request Aug 28, 2025
# Description of change

Adds a job which ensures that PR release notes are correct. It also now
enforces that release notes have no empty check boxes, so in the case
where there are no notes, the section should be deleted.

---------

Co-authored-by: Thibault Martinez <thibault@iota.org>
Signed-off-by: Roman Overko <roman.overko@iota.org>
roman1e2f5p8s pushed a commit that referenced this pull request Aug 28, 2025
# Description of change

Adds a job which ensures that PR release notes are correct. It also now
enforces that release notes have no empty check boxes, so in the case
where there are no notes, the section should be deleted.

---------

Co-authored-by: Thibault Martinez <thibault@iota.org>
roman1e2f5p8s pushed a commit that referenced this pull request Aug 28, 2025
# Description of change

Adds a job which ensures that PR release notes are correct. It also now
enforces that release notes have no empty check boxes, so in the case
where there are no notes, the section should be deleted.

---------

Co-authored-by: Thibault Martinez <thibault@iota.org>
roman1e2f5p8s pushed a commit that referenced this pull request Sep 2, 2025
# Description of change

Adds a job which ensures that PR release notes are correct. It also now
enforces that release notes have no empty check boxes, so in the case
where there are no notes, the section should be deleted.

---------

Co-authored-by: Thibault Martinez <thibault@iota.org>
roman1e2f5p8s pushed a commit that referenced this pull request Sep 2, 2025
# Description of change

Adds a job which ensures that PR release notes are correct. It also now
enforces that release notes have no empty check boxes, so in the case
where there are no notes, the section should be deleted.

---------

Co-authored-by: Thibault Martinez <thibault@iota.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Issues related to our CI pipeline dev-tools Issues related to the Developer Tools Team sc-platform Issues related to the Smart Contract Platform group.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[release-notes-generation] Revert line wrapping in commit messages

6 participants