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

fix npm publish package #112

Merged
merged 1 commit into from
Jan 1, 2021
Merged

fix npm publish package #112

merged 1 commit into from
Jan 1, 2021

Conversation

mandolyte
Copy link
Contributor

@mandolyte mandolyte commented Jan 1, 2021

After reading this: yarnpkg/yarn#685

I experimented using the yarn pack command. It now produces a package that only contains 48 files/directories instead of the thousands it had before.

The changes are these:

  • changes to the syntax used in .npmignore
  • additions to the same (in order to fit well with .gitignore)
  • removal of the "files" section in package.json

I do not know why the last one exists in the first place. So you may want to publish a beta first and what ends up in NPM.


This change is Reviewable

@RobH123 RobH123 merged commit ff13706 into master Jan 1, 2021
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.

2 participants