-
Notifications
You must be signed in to change notification settings - Fork 108
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
Prepare for/export to npmjs.com #31
Comments
NPM naming conventions are lowercase, with hyphen as separators. At a minimum, I think we need to publish two packages:
antlr4tsNote: It seems a bit unclear if we should include the antlr4ts-cliObviously, the Even though the package will have a suffix, the command name won't need to include it. Do we want to make the antlr4ts command run the TypeScript compiler on the generated files? (I think the answer is yes. We should set it up so that the global install will respect a project-local installed version (if present) when selecting which Other notes
|
Most of this is now ready, and w/ #244 I'm bumping the version number (manually) to 0.4.0. Plz review. |
There are some change we should probably make in the
project.json
file so thatnpm
will know about our package. Once that is done, I think we'll be able to usenpm link
to simplify referring to the package for debugging, samples, etc. And theimport
directives can avoid some or all of the..\
prefixes.I've never done this before, so learning curve is probably about the same for either of us.
The text was updated successfully, but these errors were encountered: