-
Notifications
You must be signed in to change notification settings - Fork 5
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
start using flow #432
base: master
Are you sure you want to change the base?
start using flow #432
Conversation
zemirco
commented
Feb 19, 2018
- add flow to the project. first component is the button
- replace standard with eslint
- update stylelint and fix all issues
|
||
[options] | ||
|
||
[strict] |
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.
[ignore]
.*/node_modules/.*/node_modules/md-components
[include]
[libs]
[lints]
[options]
[strict]
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.
in Windows it just works that way
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.
we could also activate
[options]
module.use_strict=true
server.max_workers=20
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.
what do those options do? how many workers is default?
It seems that this "flow" pull request gets a bit polluted by the stylelint required CSS changes. |