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(node): use webpack compilation hooks to emit package.json #10229

Conversation

skrtheboss
Copy link
Contributor

Use the webpack plugin system to create the package.json, this allows for developers to use plugins like
CleanWebpackPlugin which clears the dist before the compilation starts. If this plugin is used in
combination with the old method of generating the package.json, then it is deleted on compilation start.

@vercel
Copy link

vercel bot commented May 10, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
nx-dev ⬜️ Ignored (Inspect) Jun 7, 2022 at 7:16AM (UTC)

@skrtheboss skrtheboss force-pushed the fix/use-compilation-hooks-for-emitting-package-json branch 2 times, most recently from 92cb8ff to 26922df Compare May 10, 2022 07:18
@skrtheboss skrtheboss force-pushed the fix/use-compilation-hooks-for-emitting-package-json branch from 26922df to e17f7c8 Compare May 18, 2022 06:56
@skrtheboss skrtheboss force-pushed the fix/use-compilation-hooks-for-emitting-package-json branch 2 times, most recently from 0ddb3e3 to 6f569a1 Compare May 25, 2022 09:19
@jaysoo
Copy link
Member

jaysoo commented Jun 2, 2022

Thanks for the PR, we'll review it to see if this is the right approach.

@nartc
Copy link
Contributor

nartc commented Jun 4, 2022

@skrtheboss thank you for the PR. I tested this locally and it looks good. Can you rebase with latest and resolve the merge conflict?

Use the webpack plugin system to create the package.json, this allows for developers to use plugins like
CleanWebpackPlugin which clears the dist before the compilation starts. If this plugin is used in
combination with the old method of generating the package.json, then it is deleted on compilation start.
@skrtheboss skrtheboss force-pushed the fix/use-compilation-hooks-for-emitting-package-json branch from 6f569a1 to de59f0d Compare June 7, 2022 07:15
@skrtheboss
Copy link
Contributor Author

Hi @nartc, I have solved the merge conflict. Let me know if you need me to do something else.

@nartc nartc merged commit 7418ae1 into nrwl:master Jun 8, 2022
@skrtheboss skrtheboss deleted the fix/use-compilation-hooks-for-emitting-package-json branch June 8, 2022 18:43
@github-actions
Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants