Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: esModule option issue fix #476

Merged
merged 2 commits into from
Aug 5, 2020
Merged

fix: esModule option issue fix #476

merged 2 commits into from
Aug 5, 2020

Conversation

dangreen
Copy link
Contributor

@dangreen dangreen commented Aug 5, 2020

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

#475

Breaking Changes

Nope

Additional Info

...

@jsf-clabot
Copy link

jsf-clabot commented Aug 5, 2020

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented Aug 5, 2020

Codecov Report

Merging #476 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #476   +/-   ##
=======================================
  Coverage   98.68%   98.68%           
=======================================
  Files           5        5           
  Lines         228      228           
  Branches       96       96           
=======================================
  Hits          225      225           
  Misses          3        3           
Impacted Files Coverage Δ
src/index.js 93.75% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae7d211...e81969e. Read the comment docs.

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Maybe we just create:

const runtimeOptions = { attributes: options.attributes, ... }

here

?

@dangreen
Copy link
Contributor Author

dangreen commented Aug 5, 2020

@evilebottnawi done

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Thanks!

@alexander-akait alexander-akait merged commit c623f27 into webpack-contrib:master Aug 5, 2020
@dangreen dangreen deleted the fix/475 branch August 5, 2020 16:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants