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

Enabled use with NPM #102

Closed
wants to merge 2 commits into from
Closed

Enabled use with NPM #102

wants to merge 2 commits into from

Conversation

gfortaine
Copy link

@gfortaine gfortaine commented Oct 13, 2020

@gfortaine gfortaine changed the title Added a package.json so this project can be used in a NPM managed env… Enabled use with NPM Oct 13, 2020
@abenevaut
Copy link

Hello! Thanks for the PR, i'm very interested to be able to use this stuff as npm vendor.
@gfortaine could you please add detailed description to allow maintenairs (@nayuki @scrpi @0-wiz-0) to correctly understand which need you fill with this PR, and (if required) explains actions that maintenairs will have to do to publish the package (maybe advice a workflow to publish the package automatically on npm).

@@ -0,0 +1,31 @@
{
"name": "@gfortaine/qr-code-generator",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gfortaine namespace should probably decided with maintenairs.

@nayuki
Copy link
Owner

nayuki commented Aug 1, 2021

Sorry, committing thousands of lines of auto-generated, unauditable configuration files (.pnp.js, yarn.lock) is unacceptable.

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.

None yet

3 participants