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

Update assertions.js #546

Merged
merged 1 commit into from Nov 2, 2015
Merged

Update assertions.js #546

merged 1 commit into from Nov 2, 2015

Conversation

@Pklong
Copy link
Contributor

Pklong commented Nov 2, 2015

removed 'out of range' reference error, issue #545

removed 'out of range' reference error
@keithamus
Copy link
Member

keithamus commented Nov 2, 2015

Great work @Pklong! High five !

keithamus added a commit that referenced this pull request Nov 2, 2015
Update assertions.js
@keithamus keithamus merged commit 882c9cc into chaijs:master Nov 2, 2015
1 check was pending
1 check was pending
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@varjmes
Copy link

varjmes commented Nov 2, 2015

Nice one! 👋 Thanks from Your First PR (if that's how you found this bug!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.