Skip to content

Create configuration to automatically generate release notes #101375

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 2 commits into from
Apr 5, 2022
Merged

Create configuration to automatically generate release notes #101375

merged 2 commits into from
Apr 5, 2022

Conversation

itsjustkevin
Copy link
Contributor

@itsjustkevin itsjustkevin commented Apr 5, 2022

Github provides an API for automatically generating release notes.

The structure of these release notes can be configured with a yml file.

This PR is experimental and a step toward creating better release notes.

This is a work in progress and subject to change as we understand how we want to organize the information provided.

@christopherfujino
Copy link
Contributor

So if I'm reading https://docs.github.com/en/repositories/releasing-projects-on-github/automatically-generated-release-notes right, merging this will NOT generate any releases, but allow an option when manually creating a GitHub release to generate the notes?

@itsjustkevin
Copy link
Contributor Author

itsjustkevin commented Apr 5, 2022

@christopherfujino no releases will be generated. This will allow us to utilize the API to generate release notes based on the parameters specified in the config file.

When using the github UI in the release section, it will utilize the file i've created but only generate the release notes, not post them. When using the rest API, this config file can be specified as a parameter to generate notes.

Copy link
Contributor

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM

@itsjustkevin itsjustkevin merged commit fbdbb09 into flutter:master Apr 5, 2022
- engine-flutter-autoroll
- fluttergithubbot
labels:
- passed first triage
Copy link
Member

Choose a reason for hiding this comment

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

This will exclude PRs that have the labels passed first triage / passed secondary triage?

Copy link
Member

Choose a reason for hiding this comment

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

I don't think any PRs would have these labels, only issues.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@devoncarew @jmagman correct, this is still experimentation.

@devoncarew
Copy link
Member

When using the rest API, this config file can be specified as a parameter to generate notes.

Ah, cool - I didn't see there was a REST API - I thought this file just controlled the text for when you create a release (having an API definitely makes this github provided functionality more useful).

We can always consider writing our own tool here, to gather PRs or commits since the last release, and emit a markdown file for our changelog.

- tool
- title: MacOS
labels:
- platform-mac
Copy link
Member

Choose a reason for hiding this comment

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

Did you figure out if these are these processed in order? What happens to PR with tool, platform-mac, passed first triage and passed secondary triage?

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 am still working this issue, It isn't reading from the release configs the way I want yet. Will update.

- engine-flutter-autoroll
- fluttergithubbot
labels:
- passed first triage
Copy link
Member

Choose a reason for hiding this comment

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

I don't think any PRs would have these labels, only issues.

@itsjustkevin
Copy link
Contributor Author

When using the rest API, this config file can be specified as a parameter to generate notes.

Ah, cool - I didn't see there was a REST API - I thought this file just controlled the text for when you create a release (having an API definitely makes this github provided functionality more useful).

We can always consider writing our own tool here, to gather PRs or commits since the last release, and emit a markdown file for our changelog.

@devoncarew We might even be able to avoid doing that by using actions from the github market place like Release-drafter

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.

4 participants