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

⚡ Support webpack 5 #26

Merged
merged 1 commit into from
Feb 14, 2021
Merged

⚡ Support webpack 5 #26

merged 1 commit into from
Feb 14, 2021

Conversation

RDIL
Copy link
Contributor

@RDIL RDIL commented Feb 10, 2021

Signed-off-by: Reece Dunham me@rdil.rocks

Summary

  • Add support for webpack 5
  • Drop support for webpack 3, its quite old and I don't know anybody that uses it anymore
  • Other minor fixups

I've also already published a package based on my fork, so if you don't want to merge this that is fine.

Why

webpack 5 is the next big upgrade for facebook/docusaurus, so this is needed.

Checklist

  • Your code builds clean without any errors or warnings
  • You are using approved terminology
  • You have added unit tests, if apply.

Emojis for categorizing pull requests:

⚡️ New feature (:zap:)
🐛 Bug fix (:bug:)
🔥 P0 fix (:fire:)
✅ Tests (:white_check_mark:)
🚀 Performance improvements (:rocket:)
🖍 CSS / Styling (:crayon:)
♿ Accessibility (:wheelchair:)
🌐 Internationalization (:globe_with_meridians:)
📖 Documentation (:book:)
🏗 Infrastructure / Tooling / Builds / CI (:building_construction:)
⏪ Reverting a previous change (:rewind:)
♻️ Refactoring (like moving around code w/o any changes) (:recycle:)
🚮 Deleting code (:put_litter_in_its_place:)

Signed-off-by: Reece Dunham <me@rdil.rocks>
@@ -36,7 +36,7 @@
"stats": "NODE_ENV=development webpack --profile --json > compilation-stats.json"
},
"engines": {
"node": ">=4.2.4"
"node": ">=10.13.0"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

To comply with webpack's required version

@themgoncalves
Copy link
Owner

@RDIL thanks for the merge request, I will be handling it over the weekend

Copy link
Owner

@themgoncalves themgoncalves left a comment

Choose a reason for hiding this comment

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

LGTM.

Will be releasing a major version, as the drop of Webpack v3 support is a breaking change.

@themgoncalves themgoncalves merged commit 1945da2 into themgoncalves:master Feb 14, 2021
@themgoncalves
Copy link
Owner

@RDIL v1.0.0 released.

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

Successfully merging this pull request may close these issues.

None yet

2 participants