Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Web: Handle booleans for options.minify, options.style, options.style.extract #601

Merged
merged 4 commits into from
Jan 15, 2018

Conversation

timkelty
Copy link
Contributor

@timkelty timkelty commented Dec 12, 2017

This allows users to completely disable @neutrinojs/minify, @neutrinojs/style by passing false to their respective options.

Also included is the coercion of options.style.extract to {}, when passed true.

@timkelty timkelty changed the title Web: Allow overriding of options.minify Web: Allow booleans for options.minify, options.minify.*, options.style, options.style.extract Dec 12, 2017
@timkelty timkelty changed the title Web: Allow booleans for options.minify, options.minify.*, options.style, options.style.extract Web: Allow booleans for options.minify, options.style, options.style.extract Dec 13, 2017
@timkelty timkelty changed the title Web: Allow booleans for options.minify, options.style, options.style.extract Web: Handle booleans for options.minify, options.style, options.style.extract Dec 13, 2017
@timkelty
Copy link
Contributor Author

Holler if theres anything more I can do to get this merged! 💯

@timkelty timkelty requested a review from a team January 10, 2018 16:34
@timkelty
Copy link
Contributor Author

Would appreciate any eyes from @mozilla-neutrino/core-contributors to move this along and get it merged!

Copy link
Member

@eliperelman eliperelman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@eliperelman eliperelman merged commit e52919b into master Jan 15, 2018
timkelty added a commit to timkelty/neutrino that referenced this pull request Jan 17, 2018
….extract (neutrinojs#601)

* Allow overriding of `options.minify`.

* Only add in production

* Simplify extract defaults.

* Fix tests, reverse merge.
@edmorley edmorley deleted the web-override-minify branch January 30, 2018 18:18
@edmorley edmorley removed their request for review January 30, 2018 18:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

2 participants