Skip to content
This repository has been archived by the owner on Sep 9, 2021. It is now read-only.

Fix: remove webpack sourceURL #289

Merged
merged 1 commit into from
Sep 22, 2020
Merged

Conversation

cap-Bernardito
Copy link
Member

@cap-Bernardito cap-Bernardito commented Sep 22, 2020

This PR contains a:

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

Motivation / Use-Case

remove webpack sourceURL for "devtool" option ("eval-source-map" value)

Breaking Changes

No

Additional Info

No

@codecov
Copy link

codecov bot commented Sep 22, 2020

Codecov Report

Merging #289 into master will increase coverage by 0.48%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #289      +/-   ##
==========================================
+ Coverage   75.86%   76.35%   +0.48%     
==========================================
  Files           6        6              
  Lines         145      148       +3     
  Branches       52       52              
==========================================
+ Hits          110      113       +3     
  Misses         30       30              
  Partials        5        5              
Impacted Files Coverage Δ
src/supportWebpack4.js 87.50% <100.00%> (+0.83%) ⬆️
src/supportWebpack5.js 82.75% <100.00%> (+0.61%) ⬆️
src/utils.js 86.48% <100.00%> (+0.37%) ⬆️

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 0d4624c...27a9851. Read the comment docs.

@alexander-akait alexander-akait merged commit eef2757 into master Sep 22, 2020
@alexander-akait alexander-akait deleted the fix-remove-sourceURL-webpack branch September 22, 2020 13:27
TheLD6978 pushed a commit to TheLD6978/worker-loader that referenced this pull request Apr 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants