-
Notifications
You must be signed in to change notification settings - Fork 9
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
Allow multiple attributes to be specified #12
Comments
As suggested by @mojavelinux we should switch to https://www.npmjs.com/package/yargs |
ggrossetie
added a commit
to ggrossetie/asciidoctor-cli.js
that referenced
this issue
Apr 14, 2018
ggrossetie
added a commit
to ggrossetie/asciidoctor-cli.js
that referenced
this issue
Apr 21, 2018
ggrossetie
added a commit
that referenced
this issue
Apr 28, 2018
Resolves #12, replace cli by yargs
👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It's possible to specify multiple attributes, but it's not consistent with how the asciidoctor commandline works. Right now, you have to separate the attributes by spaces:
Ideally, it should be possible to specify the
-a
flag multiple times:However, that depends on array support in the cli package (see node-js-libs/cli#62).
As an alternative, we should allow attributes to be separated by commas so that quotes are not needed:
(this syntax used to be possible in Asciidoctor core, but was dropped).
If no change is made to the code, we should at least document in the README how to specify multiple attributes since it is different from the
asciidoctor
command.The text was updated successfully, but these errors were encountered: