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

all: update --napi-modules flag to not have a yes/no #186

Closed
wants to merge 3 commits into from
Closed

all: update --napi-modules flag to not have a yes/no #186

wants to merge 3 commits into from

Commits on Mar 22, 2017

  1. all: update --napi-modules flag to not have a yes/no 

    This updates the documentation, the error message upon module load
failure, the command line option parsing of the flag, and the way
the N-API addon tests pass the flag to node.

Re nodejs/node#11975 (comment)
Re nodejs/node#11975 (comment)
Fixes nodejs#184
Closes nodejs#186
    Gabriel Schulhof committed Mar 22, 2017
    Configuration menu
    Copy the full SHA
    6761e6e View commit details
    Browse the repository at this point in the history

  2. doc: mention experimental status of --napi-modules flag

    Gabriel Schulhof committed Mar 22, 2017
    Configuration menu
    Copy the full SHA
    6e18597 View commit details
    Browse the repository at this point in the history

  3. src: improve message regarding N-API experimental status

    Gabriel Schulhof committed Mar 22, 2017
    Configuration menu
    Copy the full SHA
    e2d0d96 View commit details
    Browse the repository at this point in the history