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

Bug 1693297 - Copy README.md on prepublish #77

Merged
merged 2 commits into from
Feb 17, 2021

Conversation

brizental
Copy link
Contributor

If the README.md is not in the glean package folder, it will not be included in the final package and as a consequence will not show up on the package npm page.

If the README.md is not in the glean package folder,
it will not be included in the final package and as a consequence
will not show up on the package npm page.
@@ -21,7 +21,8 @@
"fix": "eslint . --ext .ts,.js,.json --fix",
"build": "webpack --config webpack.config.js --mode production",
"dev": "webpack --watch --config webpack.config.js --mode development --devtool inline-source-map",
"prepare": "npm run build"
"prepublishOnly": "cp ../README.md ./README.md && npm run build",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed to prepublishOnly becase prepare also ran on npm install and that was annoying.

@brizental brizental merged commit 275aea7 into mozilla:main Feb 17, 2021
@brizental brizental deleted the 1693297-fix-readme branch February 17, 2021 14:19
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

2 participants