-
-
Notifications
You must be signed in to change notification settings - Fork 84
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
Removal of proposed API #675
Comments
@edgardmessias thoughts? |
This would be a breaking change and would mean a major version bump |
I agree, we can make a special plugin to use only proposed api like: https://github.com/microsoft/vscode/tree/master/extensions/git/src/api |
@edgardmessias is the revert in the diff gutter a proposed api? |
I not remember, but you can check this: https://github.com/microsoft/vscode/blob/313a14265237bc59420bc98c73fbc446b2bf38d1/src/vs/vscode.proposed.d.ts#L581 |
Yeah it will be requires |
BREAKING CHANGE: Removed proposed api functionallity (#675)
I suggest that anything using the proposed API is removed. My reasoning is that it is
Proposed API is a set of unstable API
andThey are subject to change
. Having to hack them to work is also problematic for less knowledgeable people.The text was updated successfully, but these errors were encountered: