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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Since octokit/routes is archived, should this use the OpenAPI Description instead? #77

Closed
francisfuzz opened this issue May 27, 2021 · 2 comments

Comments

@francisfuzz
Copy link

馃憢 @swinton!

I recently imported this and noticed there were some missing routes. Reading up on https://github.com/octokit/routes, it's deprecated (latest: v30.0.3) in favor of the OpenAPI Description:

https://github.com/github/rest-api-description/

I'm wondering if this repo should use that instead? Not quite sure how much work it would take to port over, but thought to raise this so others who rely on this are aware!

@francisfuzz
Copy link
Author

Update: I discovered that OpenAPI is supported by Insomnia (docs) and it's possible to import it directly. For example, the OpenAPI Description for api.github.com is available, and Insomnia will do the Right Thing:tm: by importing its raw version.

At this point, I'm wondering if we should just point folks to this method instead? 馃挱

@swinton
Copy link
Owner

swinton commented Jun 4, 2021

@imjohnbo started to PR this: #72

We felt this project was still helpful, since it also generates environment variables.

@swinton swinton closed this as completed Jun 4, 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

No branches or pull requests

2 participants