-
Notifications
You must be signed in to change notification settings - Fork 38
Feature/react transform hmre #27
Feature/react transform hmre #27
Conversation
"no-underscore-dangle": 0, | ||
"quotes": [2, "single"], | ||
"semi": [2, "always"], | ||
"space-before-function-paren": 0 |
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.
Not a fan of spaces between function
and the opening paren, so would prefer "never" here.
Oliver 👍 Looks great, dropped a few comments for things that need attention. Once that's fixed up, I'd be happy to merge this. Otherwise, wonderful. Thanks so much! |
@@ -0,0 +1 @@ | |||
v5 |
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.
Don't think this should be committed, probably should be ignored in your local .gitignore.
describe('Application', function() { | ||
it('displays the component', function() { | ||
describe('Application', function () { | ||
it('displays the component', function () { |
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.
Can you please remove these leading spaces before each function so that eslint passes? There are a few more throughout the PR too.
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.
Sorry about that... fixed now through the awesome power of npm run lint:js -- --fix
:)
Addresses #24
Upgraded all deps to bleeding edge, moved from
react-hot-loader
to Babel 6 plusreact-hmre
preset, added a small example of a stateless function