-
Notifications
You must be signed in to change notification settings - Fork 987
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
Fixes #16286 - Move multiselect from gem to npm #3786
Conversation
@@ -0,0 +1,2 @@ | |||
require('multiselect'); |
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.
Definition for rule 'no-global-assign' was not found no-global-assign
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.
looks like hound uses an older version of eslint that doesn't support this, what do you think about removing this rule @gailsteiger ?
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.
Sure, as long as you're sure that hound is using the repository rules and not the default. We can pick it up in PR code review if anyone does this, which I hope is not likely.
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.
Done in #3787
[test] |
👍 - please rebase @tbrisker and I'll merge, ping me if you want me to look at the PR in packaging |
@dLobatog rebased, thanks |
60eca5f
to
b995641
Compare
rebased again |
Thanks - I'll take care of packaging of this one 😄 |
No description provided.