Merge pull request #360 from reagent-project/browser-npm-prod-fix

Use latest Closure for React fix

docs/0.8-upgrade.md

@@ -18,6 +18,8 @@ Using Chrome React Developer Tools with this setup will break Reagent.
 
 2. Closure optimization currently breaks certain statically created objects which are
 accessed dynamically in `ReactDOM/server`: https://github.com/facebook/react/issues/12368
+Fixed by using `[com.google.javascript/closure-compiler-unshaded "v20180319"]` ([fix commit](https://github.com/google/closure-compiler/commit/c13cf48b98477e44409dba6359246bffa95b1c7b)), will be
+the default in next ClojureScript release.
 
 ## Browser - Cljsjs
 

project.clj

@@ -26,6 +26,7 @@
 
   :profiles {:dev {:dependencies [[figwheel "0.5.15"]
                                   [doo "0.1.10"]
+                                  [com.google.javascript/closure-compiler-unshaded "v20180319"]
                                   [cljsjs/prop-types "15.6.0-0"]]
                    :source-paths ["demo" "examples/todomvc/src" "examples/simple/src" "examples/geometry/src"]
                    :resource-paths ["site" "target/cljsbuild/client" "target/cljsbuild/client-npm"]}}

test-environments/browser-npm-prod/test.sh

@@ -1,10 +1,7 @@
 #!/bin/bash
 
-echo "Failure expected: https://github.com/facebook/react/issues/12368"
-echo
-
 set -ex
 
-lein do clean, doo chrome-headless prod-test-npm once || true
+lein do clean, doo chrome-headless prod-test-npm once
 test -f target/cljsbuild/prod-test-npm/main.js
 node_modules/.bin/gzip-size target/cljsbuild/prod-test-npm/main.js