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

Add webpack prod config #7513

Merged
merged 7 commits into from
Nov 6, 2024
Merged

Add webpack prod config #7513

merged 7 commits into from
Nov 6, 2024

Conversation

gjmooney
Copy link
Contributor

@gjmooney gjmooney commented Nov 5, 2024

Continuing @jtpio's work in #7354

Adds HtmlWebpackPlugin to inject cache-busting script tag.

Fixes #7373
Fixes #7221

Copy link
Contributor

github-actions bot commented Nov 5, 2024

Binder 👈 Launch a Binder on branch gjmooney/notebook/webpack_prod

@gjmooney gjmooney marked this pull request as ready for review November 6, 2024 14:07
@jtpio
Copy link
Member

jtpio commented Nov 6, 2024

Thanks @gjmooney for picking this up!

Checking with the assets built by the releaser on CI, the ?v= cache busting parameter now seems to be inlined correctly:

image

While it was not the case until now, for example when testing with notebook 7.2.2:

image

So it looks good, also since it follows what JupyterLab does: jupyterlab/jupyterlab#9776

Copy link
Member

@jtpio jtpio left a comment

Choose a reason for hiding this comment

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

Thanks!

@jtpio jtpio merged commit 110e728 into jupyter:main Nov 6, 2024
32 checks passed
@jtpio
Copy link
Member

jtpio commented Nov 6, 2024

Cutting a new beta with this change now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants