Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: webpack/webpack-dev-server
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 50ab2f07d363ee1c30660b7c3f13cf2e100dcf82
Choose a base ref
...
head repository: webpack/webpack-dev-server
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: c9271b90b476096e1cee29fa8862c1a312fa0394
Choose a head ref
Loading
Showing 540 changed files with 75,508 additions and 13,085 deletions.
13 changes: 5 additions & 8 deletions .editorconfig
Original file line number Diff line number Diff line change
@@ -1,16 +1,13 @@
root = true
# editorconfig.org

[*]
indent_style = tab
indent_size = 4
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = true
max_line_length = 233

[*.json]
indent_style = space
indent_size = 2
end_of_line = lf
insert_final_newline = true
trim_trailing_whitespace = true

[*.md]
insert_final_newline = false
trim_trailing_whitespace = false
5 changes: 5 additions & 0 deletions .eslintignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/client
/dist
!/test/client
coverage
node_modules
50 changes: 0 additions & 50 deletions .eslintrc

This file was deleted.

70 changes: 70 additions & 0 deletions .eslintrc.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,70 @@
"use strict";

module.exports = {
extends: ["webpack", "prettier"],
parser: "@babel/eslint-parser",
parserOptions: {
sourceType: "script",
ecmaVersion: 2018,
},
env: {
node: true,
es6: true,
},
rules: {
curly: "error",
"consistent-return": "off",
"no-param-reassign": "off",
"no-underscore-dangle": "off",
"prefer-destructuring": ["error", { object: false, array: false }],
"prefer-rest-params": "off",
strict: ["error", "safe"],
"global-require": "off",
},
overrides: [
{
files: ["client-src/**/*.js"],
excludedFiles: [
"client-src/webpack.config.js",
"client-src/modules/logger/SyncBailHookFake.js",
],
parserOptions: {
sourceType: "module",
allowImportExportEverywhere: true,
},
env: {
browser: true,
},
rules: {
"import/extensions": ["error", "always"],
},
},
{
files: ["test/**/*.js"],
rules: {
"no-console": "off",
},
},
{
files: [
"test/client/**/*.js",
"test/e2e/**/*.js",
"test/fixtures/**/*.js",
"test/server/liveReload-option.test.js",
],
env: {
browser: true,
node: true,
},
},
{
files: ["examples/**/*.js"],
env: {
browser: true,
},
rules: {
"no-console": "off",
},
},
],
};
1 change: 1 addition & 0 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
* @alexander-akait @hiroppy @Loonride @snitin315 @anshumanv
1 change: 1 addition & 0 deletions .github/FUNDING.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
open_collective: webpack
54 changes: 45 additions & 9 deletions .github/ISSUE_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,15 +1,51 @@
<!-- Before creating an issue please make sure you are using the latest version of webpack-dev-server and webpack. -->
<!--
Please note that this template is not optional.
Please fill out _ALL_ fields, or your issue may be closed as "invalid."
Please do not delete this template.
Please ask questions on StackOverflow or Gitter (https://gitter.im/webpack/webpack).
General questions, how-to questions, and support requests will be closed.
-->

**Do you want to request a *feature* or report a *bug*?**
<!-- Please ask questions on StackOverflow or the webpack Gitter (https://gitter.im/webpack/webpack). Questions will be closed. -->
<!--
Please place an x, no spaces, in all [ ] that apply
Please note that we are NOT accepting FEATURE requests at this time.
-->

**What is the current behavior?**
- [ ] This is a **bug**
- [ ] This is a **modification** request

**If the current behavior is a bug, please provide the steps to reproduce.**
<!-- A great way to do this is to provide your configuration via a GitHub gist. -->
### Code

**What is the expected behavior?**
<!--
If you have a large amount of code to share which demonstrates the problem
you're experiencing, or your webpack config is very large, please provide a link
to your repository rather than pasting code. We'd also encourage you to use a
Github Gist link instead of pasting code. Otherwise, please paste relevant
short snippets below.
**If this is a feature request, what is motivation or use case for changing the behavior?**
For bugs, please do consider providing a link to a stripped-down, bare-bones
repo that can reproduce the problem you're experiencing. Many times, bugs
aren't actual bugs, but rather specific issues with loaders, plugins, or
an environment/OS. Problems with complicated or large applications will almost
always require this to be triaged.
-->

**Please mention your webpack and Operating System version.**
```js
// webpack.config.js
```

```js
// additional code, remove if not needed.
```

### Please paste the results of `webpack-cli info` and `webpack-cli version` here, and mention other relevant information

### Expected Behavior

### Actual Behavior

### For Bugs; How can we reproduce the behavior?

<!-- See https://stackoverflow.com/help/minimal-reproducible-example for information on how to create good reproductions -->

### For Features; What is the motivation and/or use-case for the feature?
44 changes: 34 additions & 10 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,15 +1,39 @@
<!-- Thanks for submitting a pull request! Please provide enough information so that others can review your pull request. -->
<!--
Thank you for submitting a pull request!
**What kind of change does this PR introduce?**
<!-- E.g. a bugfix, feature, refactoring, build related change, etc… -->
Please note that this template is not optional and all _ALL_ fields must be filled out, or your pull request may be rejected.
**Did you add or update the `examples/`?**
Please do not delete this template.
Please do remove this header to acknowledge this message.
**Summary**
<!-- Explain the **motivation** for making this change. What existing problem does the pull request solve? -->
<!-- Try to link to an open issue for more information. -->
Please place an x, no spaces, in all [ ] that apply
-->

**Does this PR introduce a breaking change?**
<!-- If this PR introduces a breaking change, please describe the impact and a migration path for existing applications. -->
- [ ] This is a **bugfix**
- [ ] This is a **feature**
- [ ] This is a **code refactor**
- [ ] This is a **test update**
- [ ] This is a **docs update**
- [ ] This is a **metadata update**

**Other information**
### For Bugs and Features; did you add new tests?

<!-- Please note that we won't approve your changes if you don't add tests. -->

### Motivation / Use-Case

<!--
What existing problem does the pull request solve?
Please explain the motivation or use-case for making this change.
If this Pull Request addresses an issue, please link to the issue.
-->

### Breaking Changes

<!--
If this PR introduces a breaking change, please describe the impact and a
potential migration path for existing applications.
-->

### Additional Info
10 changes: 10 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
version: 2
updates:
- package-ecosystem: npm
directory: "/"
schedule:
interval: daily
time: "04:00"
timezone: Europe/Berlin
open-pull-requests-limit: 10
versioning-strategy: lockfile-only
104 changes: 104 additions & 0 deletions .github/workflows/nodejs.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,104 @@
name: webpack-dev-server

on:
push:
branches:
- master
- next
- v4
pull_request:
branches:
- master
- next
- v4

jobs:
lint:
name: Lint - ${{ matrix.os }} - Node v${{ matrix.node-version }}

env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

strategy:
matrix:
os: [ubuntu-latest]
node-version: [12.x]

runs-on: ${{ matrix.os }}

steps:
- uses: actions/checkout@v2
with:
fetch-depth: 0

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node-version }}
cache: "npm"

- name: Use latest NPM
run: sudo npm i -g npm

- name: Install dependencies
run: npm ci

- name: Lint
run: npm run lint

- name: Security audit
run: npm audit --production

- name: Check commit message
uses: wagoid/commitlint-github-action@v4

test:
name: Test - ${{ matrix.os }} - Node v${{ matrix.node-version }}, Webpack ${{ matrix.webpack-version }}

strategy:
matrix:
os: [ubuntu-latest, windows-latest, macos-latest]
node-version: [12.x, 14.x, 16.x]
webpack-version: [4, latest]

runs-on: ${{ matrix.os }}

steps:
- uses: actions/checkout@v2

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node-version }}
cache: "npm"

- name: Use latest NPM on ubuntu/macos
if: matrix.os == 'ubuntu-latest' || matrix.os == 'macos-latest'
run: sudo npm i -g npm

- name: Use latest NPM on windows
if: matrix.os == 'windows-latest'
run: npm i -g npm

- name: Install dependencies
run: npm ci

- name: Install webpack ${{ matrix.webpack-version }}
if: matrix.webpack-version == '4'
run: npm i webpack@${{ matrix.webpack-version }} --save-dev --ignore-scripts

- name: Link webpack-dev-server
run: |
cp -R client tmp-client
npm link --ignore-scripts
npm link webpack-dev-server --ignore-scripts
rm -r client
cp -R tmp-client client
- name: Run tests for webpack version ${{ matrix.webpack-version }}
run: npm run test:coverage -- --ci

- name: Submit coverage data to codecov
uses: codecov/codecov-action@v2
with:
token: ${{ secrets.CODECOV_TOKEN }}
Loading