Skip to content

Conversation

@octokitbot
Copy link
Contributor

If there are any new endpoints, then the code changes will likely required an update of @octokit/types. It should have received a pull request as well, merge & release it first. The @octokit/types should be updated automatically shortly after in this pull request.

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 or code changes, use a docs prefix
  • If there are typescript changes but no code changes, use fix(typescript) prefix
  • If there are code changes, use fix if a problem was resolved, feat if new endpoints / parameters were added, and feat(deprecation) if a method was deprecated.

@gr2m gr2m changed the title 🚧 GitHub OpenAPI specifications changed fix: octokit.rest.packages.listPackagesForUser uses correct route (GET /user/{username}/packages -> GET /users/{username}/packages) Sep 22, 2021
@gr2m gr2m merged commit f94bb13 into master Sep 22, 2021
@gr2m gr2m deleted the openapi-update branch September 22, 2021 18:19
@github-actions
Copy link
Contributor

🎉 This PR is included in version 5.11.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

>;
defaults: RequestInterface["defaults"];
endpoint: EndpointInterface<{ url: string }>;
};
Copy link
Contributor

Choose a reason for hiding this comment

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

Should not have been removed, I'm looking into it

@gr2m gr2m changed the title fix: octokit.rest.packages.listPackagesForUser uses correct route (GET /user/{username}/packages -> GET /users/{username}/packages) feat: octokit.rest.repos.disableLfsForRepo(), octokit.rest.repos.enableLfsForRepo(). Fix: octokit.rest.packages.listPackagesForUser uses correct route (GET /user/{username}/packages -> GET /users/{username}/packages) Sep 22, 2021
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.

3 participants