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

Support projects #457

Draft
wants to merge 16 commits into
base: master
Choose a base branch
from
Draft

Conversation

gdevanla
Copy link

No description provided.

@gdevanla
Copy link
Author

gdevanla commented Mar 14, 2021

@phadej For my personal projects I am using the /projects endpoints which is still in Github preview mode (inertia). Therefore, I have decided to extend the support here. Is this something you will be interested in accepting as a PR?

@phadej
Copy link
Contributor

phadej commented Mar 14, 2021

I'm sorry but I won't accept preview API endpoints.

Screenshot from 2021-03-14 15-02-40

Please contact GitHub developers to figure out when things are considered stable enough. I'll be happy to accept this PR after the feature is "graduated". It looks good otherwise.

@AlistairB
Copy link

FWIW, a lot of the github preview APIs are quite stable. For example, the projects api has been out for 4.5 years. Apparently it was only updated once about a month after release.

@phadej
Copy link
Contributor

phadej commented May 14, 2021

@AlistairB so what? Why it's still in preview and not "graduated" for 4.5 years. We don't know.

@AlistairB
Copy link

I guess I'm just saying a lot of the preview APIs are quite stable ie. not changed for multiple years, as you listed that as why you don't want the PR. Not saying you should support them, just providing info on the API stability thing.

Regardless, it is poor form from GitHub I agree, I don't understand how something can be in preview and unchanged for 4 years 😕

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