Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Update to be consistent with latest :optimizations default of :none #381

Closed
mfikes opened this issue Mar 9, 2015 · 0 comments
Closed

Update to be consistent with latest :optimizations default of :none #381

mfikes opened this issue Mar 9, 2015 · 0 comments
Milestone
1.0.6

Comments

@mfikes
Copy link
Contributor

mfikes commented Mar 9, 2015

With this commit, the ClojureScript compiler has been updated to default to :none for :optimizations. But, owing to this setting, cljsbuild will supply :whitespace if :optimizations is omitted.

This ticket requests that lein cljsbuild delegate to the ClojureScript compiler for the default value for this compiler option.
cemerick added a commit that referenced this issue May 4, 2015
@cemerick
Merge pull request #382 from mfikes/master
cad584a 
Fixes #381 Honor :optimizations default :none
@cemerick cemerick added this to the 1.1.0 milestone May 4, 2015
kommen added a commit to kommen/clojurescript-site that referenced this issue Mar 16, 2018
@kommen
Remove ancient, lein-cljs build version specific note in compiler opt…
383e948 
…ions

This has been fixed in lein-cljs for a while
emezeske/lein-cljsbuild#381
@kommen kommen mentioned this issue Mar 16, 2018
Merged
kommen added a commit to kommen/clojurescript-site that referenced this issue Mar 16, 2018
@kommen
Remove ancient, lein-cljsbuild version specific note in compiler options
8ddbb4b 
This has been fixed in lein-cljsbuild for a while
emezeske/lein-cljsbuild#381
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.0.6
Development

No branches or pull requests
2 participants
@cemerick @mfikes