-
Notifications
You must be signed in to change notification settings - Fork 52
Add plugins #30
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
base: main
Are you sure you want to change the base?
Add plugins #30
Conversation
|
@leebyron Lee, would you care to review this and provide feedback? Thanks. |
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.
Sorry for not seeing this change before and thank you for this contribution! This project has been a bit sleepy over the last year or so.
If you're still interested in pursuing it, I've left some comments
| formatText, | ||
| escape, | ||
| escapeCode, | ||
| updateBiblio, | ||
| getTerms, | ||
| anchorize, | ||
| join |
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.
These are implementation details that might change in the future - I'd suggest avoiding exporting them and directly relying on them
| var fs = require('fs'); | ||
| var path = require('path'); | ||
| var specmd = require('../'); | ||
| var yargs = require('yargs'); |
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.
Could you please remove yargs and use the existing simplified argument parsing? My attempt to keep dependencies low and simple
This PR is a first pass at adding plugins to spec-md which allow for the generation of output in formats other than HTML.