Skip to content

Removes "global-style" setting for npm#9256

Merged
epixa merged 1 commit intoelastic:masterfrom
epixa:revert-npm-globalstyle
Nov 29, 2016
Merged

Removes "global-style" setting for npm#9256
epixa merged 1 commit intoelastic:masterfrom
epixa:revert-npm-globalstyle

Conversation

@epixa
Copy link
Copy Markdown
Contributor

@epixa epixa commented Nov 29, 2016

This change, which was originally introduced in #9136, resulted in much
larger installs for Kibana (50MB larger in dev mode), which isn't a
reasonable tradeoff for the convenience that global-style provides.

This change, which was originally introduced in elastic#9136, resulted in much
larger installs for Kibana (50MB larger in dev mode), which isn't a
reasonable tradeoff for the convenience that global-style provides.
Copy link
Copy Markdown
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

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

LGTM

@epixa epixa merged commit 4d6e0d7 into elastic:master Nov 29, 2016
@epixa epixa deleted the revert-npm-globalstyle branch November 29, 2016 20:43
elastic-jasper added a commit that referenced this pull request Nov 29, 2016
Backports PR #9256

**Commit 1:**
Removes "global-style" setting for npm

This change, which was originally introduced in #9136, resulted in much
larger installs for Kibana (50MB larger in dev mode), which isn't a
reasonable tradeoff for the convenience that global-style provides.

* Original sha: bdf4c68
* Authored by Court Ewing <court@epixa.com> on 2016-11-29T20:29:43Z
elastic-jasper added a commit that referenced this pull request Nov 29, 2016
Backports PR #9256

**Commit 1:**
Removes "global-style" setting for npm

This change, which was originally introduced in #9136, resulted in much
larger installs for Kibana (50MB larger in dev mode), which isn't a
reasonable tradeoff for the convenience that global-style provides.

* Original sha: bdf4c68
* Authored by Court Ewing <court@epixa.com> on 2016-11-29T20:29:43Z
epixa pushed a commit that referenced this pull request Nov 29, 2016
Backports PR #9256

**Commit 1:**
Removes "global-style" setting for npm

This change, which was originally introduced in #9136, resulted in much
larger installs for Kibana (50MB larger in dev mode), which isn't a
reasonable tradeoff for the convenience that global-style provides.

* Original sha: bdf4c68
* Authored by Court Ewing <court@epixa.com> on 2016-11-29T20:29:43Z
epixa pushed a commit that referenced this pull request Nov 29, 2016
Backports PR #9256

**Commit 1:**
Removes "global-style" setting for npm

This change, which was originally introduced in #9136, resulted in much
larger installs for Kibana (50MB larger in dev mode), which isn't a
reasonable tradeoff for the convenience that global-style provides.

* Original sha: bdf4c68
* Authored by Court Ewing <court@epixa.com> on 2016-11-29T20:29:43Z
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
Backports PR elastic#9256

**Commit 1:**
Removes "global-style" setting for npm

This change, which was originally introduced in elastic#9136, resulted in much
larger installs for Kibana (50MB larger in dev mode), which isn't a
reasonable tradeoff for the convenience that global-style provides.

* Original sha: bdf4c68
* Authored by Court Ewing <court@epixa.com> on 2016-11-29T20:29:43Z

Former-commit-id: 9b43bdc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants