Skip to content

Format release notes with github script #1443

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

Merged
merged 1 commit into from
Apr 7, 2025

Conversation

klutchell
Copy link
Contributor

@klutchell klutchell commented Apr 2, 2025

The previous function was throwing errors in some cases when the gh cli expected a local .git directory.

Instead we should avoid an interactive CLI for steps like this.

Tested with both custom release notes (below) and Renovate-style release notes here.

Release Notes

Format release notes with github script to avoid interactive CLI errors at runtime.

Copy link
Contributor

flowzone-app bot commented Apr 2, 2025

Website deployed to CF Pages, 👀 preview link https://317ebc44.flowzone.pages.dev

@klutchell klutchell force-pushed the kyle/format-release-notes branch from 62cea49 to 9ce7e67 Compare April 2, 2025 17:41
@klutchell klutchell force-pushed the kyle/format-release-notes branch from 9ce7e67 to b083205 Compare April 2, 2025 17:51
@klutchell klutchell force-pushed the kyle/format-release-notes branch from b083205 to f7e5be7 Compare April 2, 2025 18:09
Copy link
Contributor

flowzone-app bot commented Apr 2, 2025

#release-notes Format release notes with github script

Notable changes

  • [only keep the important and rephrase, leaving this in place will avoid posting release notes]
  • Update src digest to 33738a0 [balena-renovate[bot]]
product-os/policy-bot (ghcr.io/product-os/policy-bot)

v0.0.152

Compare Source

(2025-04-01)

  • Update src digest to 33738a0 [balena-renovate[bot]]

v0.0.152

Compare Source

(2025-04-01)

  • Update src digest to 33738a0 [balena-renovate[bot]]

The previous function was throwing errors in some cases
when the gh cli expected a local .git directory.

Instead we should avoid an interactive CLI for steps like this.

Change-type: patch
Signed-off-by: Kyle Harding <[email protected]>

// Handle custom release notes
const customReleaseNotesMatch = pr.body.match(/## Release Notes([\s\S]*?)(?=## |$)/i);
console.log('match:', JSON.stringify(customReleaseNotesMatch, null, 2));
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you intend to leave this here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I mean it's not UNhelpful, but we can remove it if you think it's messy.

Copy link
Contributor

Choose a reason for hiding this comment

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

It doesn't bother me, I was just wondering if it was intentional

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, those ones were

@flowzone-app flowzone-app bot merged commit 86bb397 into master Apr 7, 2025
71 checks passed
@flowzone-app flowzone-app bot deleted the kyle/format-release-notes branch April 7, 2025 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants