Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

chore(package): update google-closure-compiler to version 20180402.0.0 #2573

Merged

Conversation

acdvorak
Copy link
Contributor

@acdvorak acdvorak commented Apr 16, 2018

Fixes #2558

Closure Compiler removed the LEGACY value for --module_resolution. It now defaults to BROWSER instead.

That change was made in google/closure-compiler@c2cd433

I ran npm run test:closure on this PR and master and compared their generated .closure-tmp directories. Beyond Compare found no diffs.

Fixes #2558

Closure Compiler removed the `LEGACY` value for `--module_resolution`. It now defaults to `BROWSER` instead.

That change was made in google/closure-compiler@c2cd433
@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2573   +/-   ##
=======================================
  Coverage   98.76%   98.76%           
=======================================
  Files          98       98           
  Lines        4195     4195           
  Branches      533      533           
=======================================
  Hits         4143     4143           
  Misses         52       52

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 1331fd7...235a9b2. Read the comment docs.

@acdvorak acdvorak requested a review from lynnmercier April 16, 2018 21:37
@acdvorak acdvorak removed the request for review from lynnmercier April 16, 2018 21:47
@acdvorak acdvorak assigned aprigogin and unassigned lynnmercier Apr 16, 2018
@acdvorak acdvorak merged commit 40c11ae into master Apr 16, 2018
@acdvorak acdvorak deleted the greenkeeper/google-closure-compiler-20180402.0.0-take2 branch April 16, 2018 22:46
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.

4 participants