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.

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

Create a package.json and depend on sax #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MaybeJustJames
Copy link

@MaybeJustJames MaybeJustJames commented Oct 29, 2021

Adds a package.json file to publish to NPM. Depend on sax rather than requiring users to manually vendor sax.js into their own projects.

Closes #6

@MaybeJustJames
Copy link
Author

@cmzmasek I can't assign you as a reviewer so I'm pinging you here

@MaybeJustJames
Copy link
Author

Ping @cmzmasek

1 similar comment
@MaybeJustJames
Copy link
Author

Ping @cmzmasek

@alpae
Copy link

alpae commented Jul 3, 2024

@cmzmasek would be great if you could merge this PR and make a new release on npm. current version does not allow to easily depend and reuse phyloxml. The PR looks good from my point of view.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NPM packaging: Missing package.json
2 participants