-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
Linting #5790
Linting #5790
Conversation
d72e683
to
56094a5
Compare
Codecov Report
@@ Coverage Diff @@
## master #5790 +/- ##
==========================================
+ Coverage 63.78% 63.78% +<.01%
==========================================
Files 364 364
Lines 23111 23101 -10
Branches 2587 2587
==========================================
- Hits 14741 14735 -6
+ Misses 8355 8351 -4
Partials 15 15
Continue to review full report at Codecov.
|
const wrapper = shallow(<ExploreResultsButton {...emptyQuery} />, { | ||
context: { store }, | ||
}).dive(); | ||
expect(wrapper.state().hints).to.deep.equal([]); |
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.
Hints was unused in the component.
@@ -28,9 +28,6 @@ const defaultProps = { | |||
class ExploreResultsButton extends React.PureComponent { | |||
constructor(props) { | |||
super(props); | |||
this.state = { | |||
hints: [], |
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.
hints is unused in the component
@@ -6,6 +6,10 @@ | |||
Create Date: 2016-05-27 15:03:32.980343 | |||
|
|||
""" | |||
from __future__ import absolute_import |
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.
nit: puts these inline
from __future__ import absolute_import, division, print_function, unicode_literals
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.
Somehow it's not the standard and I think the linter would complain. Not quite sure why though.
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.
🚢
Found this https://lgtm.com/projects/g/airbnb/caravel/alerts/?mode=list and started hammering at it to see what it's all about.
Related, there's more work to do, opened an issue here: #5791