Skip to content

Conversation

@genlu
Copy link
Member

@genlu genlu commented Jun 27, 2019

New version contains completion expander support.

@ivanbasov @jinujoseph @vatsalyaagrawal

New version contains completion expander support
@genlu genlu mentioned this pull request Jun 27, 2019
3 tasks
@genlu
Copy link
Member Author

genlu commented Jun 27, 2019

@vatsalyaagrawal @jinujoseph for approval

@jinujoseph
Copy link
Contributor

@genlu has editor side changes merged already ?

@genlu
Copy link
Member Author

genlu commented Jun 27, 2019

Yes. Inserted in 16.3 p1 in this PR
https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/188352?_a=overview

This change can also target 16.3 P2, since we can't merge Roslyn expander PR until 16.3 P1 is released. But I figured that we might as well build against the version that we will be shipped with.

@RikkiGibson
Copy link
Member

Please be aware that this branch is in QB mode on our side now. @genlu

@genlu
Copy link
Member Author

genlu commented Jun 28, 2019

@jinujoseph Do we need to take this to shiproom? As discussed offline, this is an infra change and the only effect would be making our unit tests run against actual shipping editor bits.

@genlu genlu closed this Jun 28, 2019
@genlu genlu reopened this Jun 28, 2019
@genlu genlu merged commit c885442 into release/dev16.3-preview1-vs-deps Jul 1, 2019
@genlu genlu deleted the dev/gel/UpdateVsSdk branch July 1, 2019 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants