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.

Sign up for GitHub

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

Jump to bottom

use es6 module syntax #83

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

use es6 module syntax #83

wants to merge 12 commits into from

Conversation

sualko
Copy link
Contributor

@sualko sualko commented Sep 20, 2017

This is a first proposal to fix #63. There are still some todos:

  • fix intention
  • use npm packages if possible (requires rollup-plugin-node-resolve to create a browser bundle)
  • ww tests fail for unknown reason
  • npm prepublish hook to create bundles
  • maybe move all export statements to the end of a file

What do you think?

@sualko
Copy link
Contributor Author

sualko commented Oct 18, 2017

@arlolra can you look into this?

@arlolra
Copy link
Owner

arlolra commented Oct 18, 2017

I'm currently travelling but hopefully soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

use npm modules instead of bundling deps
2 participants
@sualko @arlolra