Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Remove uglifyjs-webpack-plugin from dependency #118

Merged
merged 1 commit into from Mar 3, 2020
Merged

Remove uglifyjs-webpack-plugin from dependency #118

merged 1 commit into from Mar 3, 2020

Conversation

smsunarto
Copy link
Contributor

@smsunarto smsunarto commented Feb 14, 2020

What's changing?

I was browsing through and saw #82. It seems like uglifyjs is no longer used so there's no need to have it in the dependencies anymore.

Remove uglifyjs-webpack-plugin from dependencies

What else might be impacted?

There might be something I didn't notice that was using it. I already ran yarn-check and grep to make sure though.

Checklist

  • Documentation
  • New Tests
  • Added myself to contributors table
  • Added SemVer label
  • Ready to be merged

@hipstersmoothie
Copy link
Contributor

Published PR with canary version: 1.10.10-canary.118.1702

@codecov
Copy link

codecov bot commented Feb 14, 2020

Codecov Report

Merging #118 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #118   +/-   ##
=======================================
  Coverage   84.66%   84.66%           
=======================================
  Files          47       47           
  Lines         998      998           
  Branches      139      139           
=======================================
  Hits          845      845           
  Misses        153      153

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 4362de9...3fdd983. Read the comment docs.

@hipstersmoothie hipstersmoothie added the patch create a patch release label Mar 3, 2020
@hipstersmoothie hipstersmoothie merged commit c59c367 into intuit:master Mar 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
patch create a patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants