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

fix(deps): update @octokit/plugin-paginate-rest to v5 #220

Merged
merged 2 commits into from Oct 13, 2022
Merged

Conversation

gr2m
Copy link
Contributor

@gr2m gr2m commented Oct 13, 2022

  • fix(deps): update @octokit/plugin-paginate-rest to v5
  • build(package): lock file

@ghost ghost added this to Inbox in JS Oct 13, 2022
@gr2m gr2m added the Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR label Oct 13, 2022
@ghost ghost moved this from Inbox to Maintenance in JS Oct 13, 2022
@gr2m
Copy link
Contributor Author

gr2m commented Oct 13, 2022

This PR concludes the migration to the dedicated GHEC OpenAPI spec (and api.github.com OpenAPI spec no longer including GHEC routes).

It's possible that folks see type errors if they used SCIM routes before as they are no longer typed, but they shouldn't. If anything they should stop getting type support for these. For proper GHEC support there is https://github.com/octokit/plugin-enterprise-cloud.js/

Copy link
Member

@kfcampbell kfcampbell left a comment

Choose a reason for hiding this comment

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

🚀 🚀 🚀

@gr2m gr2m merged commit 9debac2 into master Oct 13, 2022
JS automation moved this from Maintenance to Done Oct 13, 2022
@gr2m gr2m deleted the update-deps branch October 13, 2022 19:06
@github-actions
Copy link
Contributor

🎉 This PR is included in version 19.0.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR
Projects
No open projects
JS
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants