-
Notifications
You must be signed in to change notification settings - Fork 71
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
The inspect documentation is missing #148
Comments
Hi @xpe, nice catch! The docs are generated from the chai.js repo. If you want, you can submit a pull request to the chai.js repo which adds the headline. It looks like this is the file that needs updating, https://github.com/chaijs/chai/blob/master/lib/chai/utils/inspect.js. You could fix it with this:
Looking at that plugins page, it looks like some of the items have the wrong headline element. They're using h1's instead of h3's. So you could make additional fixes. Change https://github.com/chaijs/chai/blob/master/lib/chai/utils/addLengthGuard.js#L12 to:
Change https://github.com/chaijs/chai/blob/master/lib/chai/utils/getActual.js#L8 to:
Change https://github.com/chaijs/chai/blob/master/lib/chai/utils/isProxyEnabled.js#L10 to:
Change https://github.com/chaijs/chai/blob/master/lib/chai/utils/proxify.js#L13 to:
Change https://github.com/chaijs/chai/blob/master/lib/chai/utils/test.js#L14 to:
|
Thanks for the detailed look and response.. For now, I just have time to report the issue. (I don't use Chai.) I was helping someone else who was looking at it. |
Okay, no problem! @keithamus, want me to open a PR? |
For sure @aaronsofaly, sounds great! Look forwarding to merging it 😄 |
On http://chaijs.com/api/plugins/, the sidebar includes 'inspect'. However, that section does not have a section heading like the others. It only shows this:
The text was updated successfully, but these errors were encountered: