Conversation
Thanks for your interest in palantir/tslint, @mgechev! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request. |
10adc78
to
7198965
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couple of minor issues
@@ -0,0 +1,21 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
file path should be ignore-module
not ignore-pattern
@@ -0,0 +1,78 @@ | |||
/** | |||
* @license | |||
* Copyright 2015 Palantir Technologies, Inc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2017
7198965
to
f23e688
Compare
@nchen63 thanks for the review! I addressed your comments. |
@mgechev thanks! |
Fix #2116
PR checklist
What changes did you make?
Introduced a rule which warns when used imports with side-effects. For more details here #2116.
Is there anything you'd like reviewers to focus on?
Have I added the required documentation.