Skip to content

Conversation

@octokitbot
Copy link
Contributor

Make sure to update the commits so that the merge results in helpful release notes, see Merging the Pull Request & releasing a new version.

In general

  • Avoid breaking changes at all costs
  • If there are no typescript changes, use build: cache as commit message
  • If there are there are only updates, use fix: ...
  • If there are any new additions, use feat: ...
  • If there are breaking changes, keep the previous ones and deprecate them. Only if there is no other way, add BREAKING CHANGE: ... to the commit body (not subject!) to trigger a breaking change.

@octokitbot octokitbot added maintenance typescript Relevant to TypeScript users only labels Jul 23, 2021
@gr2m gr2m changed the title 🚧 OpenAPI types changed fix: POST /repos/{owner}/{repo}/releases/{release_id}/assets: name query parameter is required. head.repo is nullable in pull-request schema Jul 23, 2021
@gr2m gr2m merged commit 7856c62 into main Jul 23, 2021
@gr2m gr2m deleted the openapi-update branch July 23, 2021 17:04
@octokitbot
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

typescript Relevant to TypeScript users only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants