Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

dependencies updated #335

Closed
wants to merge 1 commit into from
Closed

Conversation

cancerberoSgx
Copy link

BTW would be nice to set up CI like travis or appveyor just to make sure PRs are not breaking the extension. I tried to debug manually the most I can but running the test of this extension is very very hard and I give up. Please implement a simple npm test command that perform most of the manual work you detail in /README.md !! You can use shelljs to copy things and execute commands programatically very easily.

@msftclas
Copy link

msftclas commented Apr 7, 2018

CLA assistant check
All CLA requirements met.

@egamma egamma added this to the April 2018 milestone Apr 9, 2018
@egamma
Copy link
Member

egamma commented Apr 13, 2018

@cancerberoSgx thanks for the PR, I will combine this updates with an update to the new Language Server libraries.

I will look into simplifying the setup and support for travis, makes total sense.

@egamma egamma modified the milestones: April 2018, May 2018 Apr 30, 2018
egamma added a commit that referenced this pull request Jun 4, 2018
@egamma
Copy link
Member

egamma commented Jun 4, 2018

@cancerberoSgx I´ve merged the PR manually and updated again to the latest versions, thanks again.

@egamma egamma closed this Jun 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants