Skip to content

Conversation

RDIL
Copy link
Contributor

@RDIL RDIL commented Mar 10, 2021

Summary

Just one more to go. This one was just 2 Array to Set changes.

Not sure why the lockfile got updated, but I also fixed a typo and some warnings outputted while running the tests.

Why

Its a warning.

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:)

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.

Thanks for the PR!

target: 'web',
entry: {
index: './example/client.jsx',
},
devtool: 'cheap-module-eval-source-map',
devtool: 'eval-cheap-module-source-map',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

For testing against webpack 5 to work, this change is required, and the other 2 fix warnings.

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

@themgoncalves themgoncalves merged commit 5d0c6a6 into themgoncalves:master Mar 15, 2021
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.

2 participants