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

Proxy support #27

Merged
merged 3 commits into from
Aug 1, 2016
Merged

Proxy support #27

merged 3 commits into from
Aug 1, 2016

Conversation

mad-mike
Copy link
Contributor

@mad-mike mad-mike commented Aug 1, 2016

implementing #25

@@ -1,4 +1,4 @@
"use strict";
/// <reference path="../../typings/vscode-typings.d.ts" />
Copy link
Owner

@Huachao Huachao Aug 1, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mad-mike I have a question that why adding this line and removing the strict mode?

Copy link
Contributor Author

@mad-mike mad-mike Aug 1, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found that the typescript compiler will introduce "use strict" anyway, so I removed that line.
Then I added the ref to the type definitions in order to get IntelliSense for the vscode API.

Copy link
Owner

@Huachao Huachao Aug 1, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mad-mike, I found even without this line, I can still get IntelliSense with vscode. And I think we can remove this line, together with the 'use strict' line

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Huachao, you are right. Explicit typings are only necessary for npm packages that don't contain them out of the box, as it seems (http://www.typescriptlang.org/docs/handbook/typings-for-npm-packages.html). I'm going to save a few lines of code from now on ;)

@Huachao Huachao merged commit 69e6462 into Huachao:master Aug 1, 2016
@Huachao
Copy link
Owner

Huachao commented Aug 1, 2016

Merged, thanks

Huachao added a commit that referenced this pull request Jan 10, 2017
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.

2 participants