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

Add support and testing for Sinon 2.x #89

Merged
merged 1 commit into from
Mar 21, 2017
Merged

Add support and testing for Sinon 2.x #89

merged 1 commit into from
Mar 21, 2017

Conversation

domenic
Copy link
Collaborator

@domenic domenic commented Mar 20, 2017

This uses Travis to test against the different versions supported.

Let's see what if this works according to Travis...

@domenic domenic force-pushed the sinon-2 branch 2 times, most recently from 1e92f24 to a7a12bd Compare March 20, 2017 04:28
@domenic
Copy link
Collaborator Author

domenic commented Mar 20, 2017

The tests are failing on Sinon 2.0.0 because of sinonjs/sinon#1342. I suppose we could add a workaround that accepts the buggy message format for now, but not this weekend...

@fstoerkle
Copy link

Sinon has released 2.1.0 which fixes this bug: https://github.com/sinonjs/sinon/blob/master/Changelog.txt :)

This uses Travis to test against the different versions supported.
@domenic domenic merged commit 46d2685 into master Mar 21, 2017
@domenic domenic deleted the sinon-2 branch March 21, 2017 10:30
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.

None yet

2 participants