-
Notifications
You must be signed in to change notification settings - Fork 154
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(publish): ensure we publish the correct files #1042
Conversation
✅ Deploy Preview for maci-typedoc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
55dc4b0
to
a88e6e3
Compare
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.
@ctrlc03 thanks, just one question: do we need different files like license, changelog, readme for npm packages?
yea think that would be good, I'll explicitly add them to files |
@0xmad seems like LICENSE gets added automatically from project root https://www.npmjs.com/package/maci-cli/v/0.0.0-ci.390bdc3?activeTab=code |
Ok, let's merge it then. |
I'll add changelog too, can't hurt having it |
a88e6e3
to
bc42724
Compare
Description
Ensure we publish the correct files by setting the correct paths inside package.json
Confirmation