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

feat: release on specific labels #101

Merged
merged 1 commit into from
Apr 30, 2024
Merged

Conversation

jmeridth
Copy link
Contributor

@jmeridth jmeridth commented Apr 29, 2024

Related to github/github-ospo#105

Pull Request

Proposed Changes

Only generate a release if our semver related labels (breaking, enhancement, fix) or the release label are present on a merged pull request.

Changed from push (merge) on main branch to release generation happening when a pull_request is merged to main branch.

This gives us access to the pull requests labels without having to make API cals.

Currently we'd still need to label a pull request with release if it is a dependabot or manual pull request related to a CVE or security fix.

  • update CONTRIBUTING.md with new release information
  • manually add vuln and release labels to repository

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run npm run lint and fix any linting issues that have been introduced
  • run npm run test and run tests

Reviewer

  • Label as either bug, documentation, enhancement, infrastructure, maintenance, or breaking

Only generate a release if our semver related labels (`breaking`, `enhancement`, `fix`) or the `release` label are present on a merged pull request.

Changed from push (merge) on main branch to release generation happening when a pull_request is merged to main branch.

This gives us access to the pull requests labels without having to make API cals.

Currently we'd still need to label a pull request with `release` if it is a dependabot or manual pull request related to a CVE or security fix.

- [x] update CONTRIBUTING.md with new release information
- [x] manually add `vuln` and `release` labels to repository

Signed-off-by: jmeridth <[email protected]>
@jmeridth jmeridth added the enhancement New feature or request label Apr 29, 2024
@jmeridth jmeridth requested a review from a team as a code owner April 29, 2024 18:52
@jmeridth jmeridth requested a review from ajhenry April 29, 2024 19:07
Copy link
Contributor

@zkoppert zkoppert left a comment

Choose a reason for hiding this comment

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

lgtm! @ajhenry Feel free to review too for if this is your desired functionality. I reviewed based on syntax and functional correctness.

Copy link
Contributor

@ajhenry ajhenry left a comment

Choose a reason for hiding this comment

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

This looks awesome! Thanks a ton for setting it up ❤️

@ajhenry ajhenry merged commit 09f3f7a into main Apr 30, 2024
14 checks passed
@ajhenry ajhenry deleted the jm-specific-labels-for-releases branch May 17, 2024 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants