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

Change module system of the main file #93

Merged
merged 2 commits into from
Apr 23, 2017
Merged

Change module system of the main file #93

merged 2 commits into from
Apr 23, 2017

Conversation

thiamsantos
Copy link
Contributor

I tried to make less changes as possible, but to solve #92 I needed to add a build process. So, if this PR be accepted, all future changes needed to be done on the file index.es.js which will compiled to index.js running npm run build.

Changelog:

  • Added rollup to compile the module.
  • Changed jshint/jscs config to make the tests pass.
  • Expose a version using es6 modules on module and jsnext:main properties of package.json.

@flitbit flitbit merged commit c62e8c0 into flitbit:master Apr 23, 2017
@thiamsantos
Copy link
Contributor Author

thiamsantos commented Apr 23, 2017

Thanks @flitbit!!!! Are you going to release these changes soon?

@thiamsantos thiamsantos deleted the esmodules branch April 23, 2017 15:44
@flitbit
Copy link
Collaborator

flitbit commented Apr 23, 2017

Working on rollup release right now.

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