Skip to content

Conversation

sam-github
Copy link
Contributor

Documented CLI of

branch-diff --simple master fix-cli-flags

doesn't work, minimist assumes master is the value of --simple option,
it needs to be told about options that don't take values.

@sam-github
Copy link
Contributor Author

@nodejs/releasers -- PTAL

Documented CLI of

   ./branch-diff.js --simple master fix-cli-flags

doesn't work, minimist assumes master is the value of `--simple` option,
it needs to be told about options that don't take values.
@sam-github
Copy link
Contributor Author

PTAL

@targos
Copy link
Member

targos commented Jun 4, 2019

/cc @MylesBorins @rvagg who have publish rights on npm

@rvagg
Copy link
Member

rvagg commented Jun 5, 2019

code style isn't the same as core here so this'll need to be fixed to match, I'll try and get around to that today

@rvagg rvagg mentioned this pull request Jun 6, 2019
@rvagg
Copy link
Member

rvagg commented Jun 6, 2019

pulled this in to #18 along with formatting fixes and some other updates

@rvagg rvagg closed this Jun 6, 2019
@sam-github sam-github deleted the fix-cli-flags branch June 6, 2019 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants