-
Notifications
You must be signed in to change notification settings - Fork 84
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
resolving cli flags to commands #104
Conversation
src/functions/flagHelper.js
Outdated
@@ -0,0 +1,47 @@ | |||
// @flow |
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.
This shouldn't be in src/functions
cc60fb4
to
0846073
Compare
0846073
to
e95b1f3
Compare
cf78377
to
6ffcdb6
Compare
@jamiebuilds @lukebatchelor request review. 🙂 |
// @flow | ||
import * as options from './options'; | ||
|
||
const BOOLEAN_FLAGS = { |
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.
I thought we discussed this before, that we couldnt have a hard coded list of boolean flags, because it's context dependent?
The approach taken here is not going to work:
|
passing flags to command. Fix : #90
Need Review. 🙂