Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(options): initial support for long options #926

Merged
merged 2 commits into from Jan 5, 2019
Merged

feat(options): initial support for long options #926

merged 2 commits into from Jan 5, 2019

Commits on Jan 4, 2019

  1. feat(options): initial support for long options 

    This adds initial support for long options. This integration tests them
with `shell.touch()` and unit tests them in `test/common.js`. This
documents the new syntax.

This also refactors some of the common internals:

 * Creates a new CommandError type to replace the `'earlyExit'` hack
 * Clearer Error types for `parseOptions()` exceptions
 * Removes side effects from a test which modifies `common.config`
 * Fixes appveyor and travis config to run lint (regressed in #920)

Issue #924
Test: touch.js, common.js
    @nfischer
    nfischer committed Jan 4, 2019
    Copy the full SHA
    a27a7c7 View commit details
    Browse the repository at this point in the history

  2. small refactor for errorOptions

    @nfischer
    nfischer committed Jan 4, 2019
    Copy the full SHA
    7a5afec View commit details
    Browse the repository at this point in the history