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 generator publicPath behavior when empty string #13858

Merged
merged 1 commit into from Jul 25, 2021

Conversation

@Janpot
Copy link
Member

@Janpot Janpot commented Jul 24, 2021

Fixing the second issue described in this PR vercel/next.js#27413 (comment)
empty string in rule.generator.publicPath gets ignored.

What kind of change does this PR introduce?

bugfix

Did you add tests for your changes?

yep

Does this PR introduce a breaking change?

nope

What needs to be documented once your changes are merged?

nothing

@webpack-bot
Copy link
Contributor

@webpack-bot webpack-bot commented Jul 24, 2021

For maintainers only:

  • This needs to be documented (issue in webpack/webpack.js.org will be filed when merged)
  • This needs to be backported to webpack 4 (issue will be created when merged)
@webpack-bot
Copy link
Contributor

@webpack-bot webpack-bot commented Jul 24, 2021

@Janpot The most important CI builds failed. This way your PR can't be merged.

Please take a look at the CI results from azure (0 errors / 0 warnings) and appveyor (failure) and fix these issues.

@sokra
sokra approved these changes Jul 25, 2021
@sokra sokra merged commit beb42c6 into webpack:main Jul 25, 2021
37 of 38 checks passed
37 of 38 checks passed
@github-actions
lint
Details
@github-actions
basic
Details
@github-actions
unit
Details
@github-actions
integration (ubuntu-latest, 10.x, a)
Details
@github-actions
integration (ubuntu-latest, 10.x, b)
Details
@github-actions
integration (ubuntu-latest, 16.x, a)
Details
@github-actions
integration (ubuntu-latest, 16.x, b)
Details
@github-actions
integration (windows-latest, 10.x, a)
Details
@github-actions
integration (windows-latest, 10.x, b)
Details
@github-actions
integration (windows-latest, 16.x, a)
Details
@github-actions
integration (windows-latest, 16.x, b)
Details
@github-actions
integration (macos-latest, 10.x, a)
Details
@github-actions
integration (macos-latest, 10.x, b)
Details
@github-actions
integration (macos-latest, 16.x, a)
Details
@github-actions
integration (macos-latest, 16.x, b)
Details
@github-actions
integration (ubuntu-latest, 14.x, a)
Details
@github-actions
integration (ubuntu-latest, 12.x, a)
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
licence/cla Contributor License Agreement is signed.
Details
@azure-pipelines
webpack.webpack Build #20210724.1 succeeded
Details
@azure-pipelines
webpack.webpack (Linux node-10-a) Linux node-10-a succeeded
Details
@azure-pipelines
webpack.webpack (Linux node-10-b) Linux node-10-b succeeded
Details
@azure-pipelines
webpack.webpack (Linux node-12-a) Linux node-12-a succeeded
Details
@azure-pipelines
webpack.webpack (Linux node-14-a) Linux node-14-a succeeded
Details
@azure-pipelines
webpack.webpack (Linux node-16-a) Linux node-16-a succeeded
Details
@azure-pipelines
webpack.webpack (Linux node-16-b) Linux node-16-b succeeded
Details
@azure-pipelines
webpack.webpack (Windows node-10-a) Windows node-10-a succeeded
Details
@azure-pipelines
webpack.webpack (Windows node-10-b) Windows node-10-b succeeded
Details
@azure-pipelines
webpack.webpack (Windows node-12-a) Windows node-12-a succeeded
Details
@azure-pipelines
webpack.webpack (Windows node-12-b) Windows node-12-b succeeded
Details
@azure-pipelines
webpack.webpack (Windows node-16-a) Windows node-16-a succeeded
Details
@azure-pipelines
webpack.webpack (Windows node-16-b) Windows node-16-b succeeded
Details
@azure-pipelines
webpack.webpack (basic) basic succeeded
Details
@azure-pipelines
webpack.webpack (lint) lint succeeded
Details
@azure-pipelines
webpack.webpack (macOS node-12-a) macOS node-12-a succeeded
Details
@azure-pipelines
webpack.webpack (macOS node-12-b) macOS node-12-b succeeded
Details
@azure-pipelines
webpack.webpack (macOS node-16-a) macOS node-16-a succeeded
Details
@azure-pipelines
webpack.webpack (macOS node-16-b) macOS node-16-b succeeded
Details
@sokra
Copy link
Member

@sokra sokra commented Jul 25, 2021

Thanks

@Janpot Janpot deleted the Janpot:genrator-publicPath-override branch Jul 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants