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 related projects into README.md #736

Merged
merged 1 commit into from
Jun 24, 2016

Conversation

lucasfcosta
Copy link
Member

As @meeber remembered here we needed to update the related projects on README.md, so here it is.
I added every project which is not a Chai plugin and is inside the ChaiJS repo, please let me know if you guys don't want to include any of them.

- [chaijs / assertion-error](https://github.com/chaijs/assertion-error): Custom `Error` constructor thrown upon an assertion failing.
- [chaijs / deep-eql](https://github.com/chaijs/deep-eql): Improved deep equality testing for Node.js and the browser.
- [chaijs / type-detect](https://github.com/chaijs/type-detect): Improved typeof detection for node.js and the browser.
- [chaijs / type-detect](https://github.com/chaijs/type-detect): Improved typeof detection for Node.js and the browser.
- [chaijs / check-error](https://github.com/chaijs/check-error): Error comparison and information related utility for NodeJS and the browser.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only nitpick I'll throw at you is changing NodeJS to Node.js on line 41 for consistency.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@meeber Thanks for noticing!
Done 😄

@lucasfcosta lucasfcosta force-pushed the readme-update-related-projects branch from a610485 to 4f0d798 Compare June 23, 2016 23:58
@meeber
Copy link
Contributor

meeber commented Jun 24, 2016

LGTM

@keithamus
Copy link
Member

LGTM!

@keithamus keithamus merged commit 9dcd0eb into chaijs:master Jun 24, 2016
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.

3 participants