Skip to content

chore(docs): fix help option#176

Merged
mthenw merged 1 commit intomthenw:masterfrom
mojoaxel:issue/53/help-option
Aug 6, 2019
Merged

chore(docs): fix help option#176
mthenw merged 1 commit intomthenw:masterfrom
mojoaxel:issue/53/help-option

Conversation

@mojoaxel
Copy link
Copy Markdown
Contributor

@mojoaxel mojoaxel commented Aug 3, 2019 •
edited
Loading

This should fix #53 after commander.js@3.0.0 is released.

see 3.0.0/CHANGELOG

@mojoaxel mojoaxel force-pushed the issue/53/help-option branch 2 times, most recently from d0400d8 to 4274395 Compare August 3, 2019 21:01
@mojoaxel mojoaxel force-pushed the issue/53/help-option branch from 4274395 to c0a2e9f Compare August 3, 2019 21:02
@mthenw
Copy link
Copy Markdown
Owner

mthenw commented Aug 6, 2019

Great stuff. I will review and test it later.

@mthenw mthenw marked this pull request as ready for review August 6, 2019 17:29
@mthenw
Copy link
Copy Markdown
Owner

mthenw commented Aug 6, 2019

I tested it and looks good.

@mthenw mthenw merged commit 364f68b into mthenw:master Aug 6, 2019
@mojoaxel
Copy link
Copy Markdown
Contributor Author

mojoaxel commented Aug 6, 2019

😟 commander.js has not yet been released, yet. This pull-request draft relies on a prerelease at the moment. It should not be released as it is at the moment.

After v3.0.0 has been released we should change the dependency and than we frontail should be released.

@mthenw
Copy link
Copy Markdown
Owner

mthenw commented Aug 6, 2019 •
edited
Loading

Yeah, I got it but I tested it and works fine. Have you spotted any errors?

@mojoaxel
Copy link
Copy Markdown
Contributor Author

mojoaxel commented Aug 6, 2019

Yeah, I got it but I tested it and works fine. Have you spotted any errors?

I'm just not sure if this 3.0.0-0 pre-release is reliable. I don't have a good feeling about this. But it works 👍

@mthenw
Copy link
Copy Markdown
Owner

mthenw commented Aug 6, 2019

OK, let's wait a few days. Maybe the release will happen in the meantime.

@mojoaxel
Copy link
Copy Markdown
Contributor Author

mojoaxel commented Aug 8, 2019

https://github.com/tj/commander.js/releases/tag/v3.0.0
@mthenw You can now update the dependency and release this 🚀

@mthenw
Copy link
Copy Markdown
Owner

mthenw commented Aug 8, 2019

It's not published to npm yet.

mojoaxel added a commit to mojoaxel/frontail that referenced this pull request Aug 30, 2019
@mojoaxel
update commander to 3.0.1
446c0ad 
see also mthenw#176
@mojoaxel mojoaxel mentioned this pull request Aug 30, 2019
Merged
mthenw pushed a commit that referenced this pull request Aug 30, 2019
@mojoaxel @mthenw
update commander to 3.0.1 (#182)
09acdc6 
see also #176
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

README lists -h option as alias for both --help and --host

2 participants

@mojoaxel @mthenw