-
Notifications
You must be signed in to change notification settings - Fork 32.5k
enableProposedApi not documented in package.json #33358
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
Comments
Yeah, that is actually on purpose because we don't want to over-promote this. Still, we should document somewhere how proposed API to be used |
IMO we should add it. The description can explain the limitations you get when you use it. Given the limitations, there's IMO no danger that too many devs will use it. |
@jrieken Can we include it in a section under Alternatively, as @rcjsuen suggested, put the instruction to the top of |
I added
@jrieken Please verify the docs and sample, and edit as you see fit. |
👍 for the documentation and 👎 for the sample. That will be out of date in no-time and I also don't think that it adds value |
@jrieken However, currently it's awkward to update your code to use Proposed API. Each time I had to remember to do this:
If sample is not the right place, what do you think about adding another script to the |
@octref step 1 to 4 have nothing to do with proposed api. that's the procedure when coding against the latest, not yet published, version of for proposed API the three steps outline here are needed: https://code.visualstudio.com/updates/v1_28#_proposed-extension-apis |
OK I'll drop the sample. |
Should this issue be reopened or marked as wont-fix? |
To verify:
|
Refs: #33248
enableProposedApi
is not suggested in package.json inteli-senseDue to that it is hard to discover this flag. Is this on purpose?
The text was updated successfully, but these errors were encountered: