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

Expose Rationalized Completion API #3538

Closed
DustinCampbell opened this issue Jun 16, 2015 · 7 comments
Closed

Expose Rationalized Completion API #3538

DustinCampbell opened this issue Jun 16, 2015 · 7 comments
Assignees
Labels
0 - Backlog Area-IDE Concept-API This issue involves adding, removing, clarification, or modification of an API. Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented
Milestone

Comments

@DustinCampbell
Copy link
Member

Currently, there is an internal API for retrieving the completion items that would be displayed in a completion list. This is not currently in a shape that is ready for public consumption and needs rationalization.

@DustinCampbell DustinCampbell added Area-IDE Concept-API This issue involves adding, removing, clarification, or modification of an API. 0 - Backlog labels Jun 16, 2015
@DustinCampbell DustinCampbell self-assigned this Jun 16, 2015
@DustinCampbell DustinCampbell added this to the 1.1 milestone Jun 16, 2015
@DustinCampbell DustinCampbell changed the title Rationalize Completion API Expose Rationalized Completion API Jun 16, 2015
@dfkeenan
Copy link

Hi @DustinCampbell ,

Just wondering how you are going with this? Every now and then I check the repository hoping there is a public API. If there is one I didn't find it. Guess I should just keep an eye on this thread instead 😉

Looking forward to putting your fine work to "good" use.

Cheers,
dfkeenan

@dfkeenan
Copy link

Pushed back to 1.3 😢

Is there any info on what/how this is going to be exposed? Even code in a branch or something would be interesting to look at.

@DustinCampbell
Copy link
Member Author

How about code in a PR? #8170

@dfkeenan
Copy link

dfkeenan commented Apr 7, 2016

😮 Quite an epic PR (as expected).

If I am reading things right shouldn't be a huge amount of work to transition from the original version. Though I'll probably scratch that project I started back when roslyn was a CTP.

I eventually want to add scripting support to an app I am working on. I have plenty to do before I "need" this though. I look forward to playing with it when it gets released. 😄

Keep up the good work.

@DustinCampbell
Copy link
Member Author

Yeah, it is a bit... epic. I think @mattwar was going to try to clean up the PR a bit to make it easier to review. 😊

@DustinCampbell DustinCampbell added the Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented label May 11, 2016
@DustinCampbell
Copy link
Member Author

This was implemented with #8170

@weltkante
Copy link
Contributor

Consuming the completion API is blocked by issue #11841 unless you are using Microsoft.VisualStudio.Composition as container (normal MEF container cannot load the CSharp.Feature assembly).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - Backlog Area-IDE Concept-API This issue involves adding, removing, clarification, or modification of an API. Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented
Projects
None yet
Development

No branches or pull requests

6 participants