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.

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

Added browserify support #51

Merged
merged 1 commit into from
Mar 2, 2017
Merged

Conversation

craigh411
Copy link
Contributor

Added the babilify transform for browserify in packge.json so the src files can be transpiled in browserify projects.

Fix or Enhancement?

Browserify applies transforms on a per-module basis and doesn't appear to allow global transforms, meaning that babelify needs to be declared in the modules package.json in order to transpile the ES2015 code in src.

In order for a browserify user to transpile the code, they would need to install babelify and babel-preset-es2015 and add a .babelrc file in the root of their project with the following code:

{ 
"presets": ["es2015"]
}

I think you could get away with explaining that rather than creating a dependency for babelify and babel-preset-es2015 because many vue/browserify users will already have their projects set up this way.

Added the `babilify` transform for `browserify` for `packge.json` so the src files can be transpiled in browserify projects.
@codecov
Copy link

codecov bot commented Mar 2, 2017

Codecov Report

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

@@          Coverage Diff          @@
##           master    #51   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           9      9           
  Lines          75     75           
=====================================
  Hits           75     75

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 1a22bae...ada0fab. Read the comment docs.

@apertureless apertureless self-requested a review March 2, 2017 14:02
@apertureless apertureless changed the base branch from master to develop March 2, 2017 18:36
@apertureless apertureless merged commit 819b96b into apertureless:develop Mar 2, 2017
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.

2 participants