Skip to content

Don't wrap in array and object completions#29

Merged
aeschli merged 3 commits intomicrosoft:masterfrom
FloEdelmann:patch-1
Oct 22, 2018
Merged

Don't wrap in array and object completions#29
aeschli merged 3 commits intomicrosoft:masterfrom
FloEdelmann:patch-1

Conversation

@FloEdelmann
Copy link
Contributor

@aeschli
Copy link
Collaborator

aeschli commented Oct 2, 2018

@FloEdelmann Please update the tests as well.

@aeschli aeschli added this to the October 2018 milestone Oct 2, 2018
@msftclas
Copy link

msftclas commented Oct 2, 2018

CLA assistant check
All CLA requirements met.

@FloEdelmann
Copy link
Contributor Author

@aeschli The tests (and another place where the same completions are suggested) are now fixed.

Unfortunately, the Travis CI tests still fail. Am I right that that is not related to my changes?

@FloEdelmann
Copy link
Contributor Author

@aeschli Any updates on this? Is it possible that this will make it into the October Release?

@aeschli aeschli merged commit 51d84b0 into microsoft:master Oct 22, 2018
@aeschli
Copy link
Collaborator

aeschli commented Oct 22, 2018

merged, thanks!

@FloEdelmann
Copy link
Contributor Author

Nice, thank you!

@FloEdelmann FloEdelmann deleted the patch-1 branch October 22, 2018 15:31
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