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
Add --exclude-script to respec2html #541
Conversation
@@ -30,6 +31,13 @@ if (args.indexOf("-w") !== -1) { | |||
reportWarnings = true; | |||
} | |||
|
|||
if (args.indexOf("--exclude-script") !== -1) { | |||
var idx = args.indexOf("--exclude-script"); | |||
var values = args.splice(idx, 2); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you certain you want "splice" here (instead of slice
)? Splice modifies the args array, which is not great IMO (i.e., we should keep inputs immutable).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've re-used here the approach taken in the two previous argument parsing blocks.
LGTM |
Doesn't this only permit one exclusion instead of a list of exclusions? |
it filters any scripts whose source starts with the url passed in parameter; I could support several of those, but that doesn't seem required yet |
👍 |
Add --exclude-script to respec2html
…rtain elements should exist on a graph, such as the plot bounds
close #538