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

chore: add npmignore to babel-helper-plugin-utils #11674

Merged

Conversation

JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Jun 4, 2020

Q                       A
License MIT

The src folder is never meant to be published.

I also checked all the @babel/* packages, only this one does not have an .npmignore.

// /tmp/check-npm-ignore.js
require("fs").statSync(require("path").resolve(process.cwd(),".npmignore"))
lerna exec --scope="@babel/*" -- node /tmp/check-npm-ignore.js

@JLHwung JLHwung added the PR: Internal 🏠 A type of pull request used for our changelog categories label Jun 4, 2020
@babel-bot
Copy link
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/23444/

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jun 4, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit a7da195:

Sandbox Source
blissful-water-x5ko8 Configuration
great-swartz-tvz3h Configuration

@nicolo-ribaudo nicolo-ribaudo merged commit fdbbb57 into babel:master Jun 4, 2020
@nicolo-ribaudo nicolo-ribaudo deleted the add-npmignore-to-helper-plugin-utils branch June 4, 2020 22:44
@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Sep 4, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Internal 🏠 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants