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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port to es6 #40

Merged
merged 6 commits into from Apr 3, 2017

Conversation

@kvz
Copy link
Member

kvz commented Jan 19, 2017

Hey @tim-kos @adrusi, not sure if you want this change, but I'm using increasingly complex projects to stress-test 馃尶 Invig and the node-sdk was a good step up in the chain.

If you don't like it, that's ok. I'll have gained more experience and invig is better because of it already, making it a slightly better fit for the next step in complexity.

PS The tests pass locally for me (but I have not run integration tests)

screenshot 2017-01-19 21 41 59

kvz added 3 commits Jan 19, 2017
@kvz

This comment has been minimized.

Copy link
Member Author

kvz commented Jan 20, 2017

@tim-kos

This comment has been minimized.

Copy link
Member

tim-kos commented Jan 26, 2017

Alright, cheers. Thanks for doing this. <3

Going to check for integration tests and then review this.

kvz and others added 3 commits Feb 17, 2017
format es6
@kvz kvz merged commit 5a61108 into master Apr 3, 2017
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
@kvz kvz deleted the es6 branch Apr 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can鈥檛 perform that action at this time.