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

Re-organise browserify transform #36

Merged
merged 1 commit into from
Feb 5, 2016
Merged

Re-organise browserify transform #36

merged 1 commit into from
Feb 5, 2016

Conversation

bassjacob
Copy link
Contributor

Move babelify transform out of browserify key in package.json. This
change is due to an error on deploy when requiring this module from a
project that uses a newer version of babel. This error is due to an
incompatability in the babelrc referencing an old key that isn't used by
babel 6.

Since this project exports the babel-ified version in the dist/ folder,
there isn't any need to run browserify or babelify over the code when using
it as a dependency in another project.

Add npmignore file to slim down production release. This file will ignore
any files not needed to use this module as a dependency.

Move babelify transform out of browserify key in package.json. This
change is due to an error on deploy when requiring this module from a
project that uses a newer version of babel. This error is due to an
incompatability in the babelrc referencing an old key that isn't used by
babel 6.

Since this project exports the babel-ified version in the dist/ folder,
there isn't any need to run browserify or babelify over the code when using
it as a dependency in another project.

Add npmignore file to slim down production release. This file will ignore
any files not needed to use this module as a dependency.
@bassjacob bassjacob changed the title Re-organise browerify transform Re-organise browserify transform Feb 5, 2016
@@ -1,13 +1,14 @@
{
"name": "react-gmaps",
"version": "1.2.2",
"name": "@subshad/react-gmaps",
Copy link
Owner

Choose a reason for hiding this comment

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

Just de-own the package and we are ready to go :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Whoops, I thought I commited that after I opened this. I'll de-own it now :)

@MicheleBertoli
Copy link
Owner

Thank you very much @subshad
I had a chat about this with my colleague @ben-davis a few weeks ago but I didn't have time to implement it yet.

@bassjacob
Copy link
Contributor Author

I think that should do it. Sorry about the naming commit

MicheleBertoli added a commit that referenced this pull request Feb 5, 2016
Re-organise browserify transform
@MicheleBertoli MicheleBertoli merged commit 250389d into MicheleBertoli:master Feb 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants