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
Document spying on accessors #1976
Conversation
ddf4936
to
668c158
Compare
Pull Request Test Coverage Report for Build 2790
💛 - Coveralls |
Pull Request Test Coverage Report for Build 2817
💛 - Coveralls |
Documentation updates are very valuable to open source projects ⭐️ The documentation for Sinon is a bit special, so you'll have to also put the changes into all the previous releases for which this applies. It's sort of hinted in https://github.com/sinonjs/sinon/blob/master/docs/CONTRIBUTING.md#example-documenting-a-fixed-bug, just not in a very obvious way. Documentation is an area I'd like to significantly improve in 2019. The feature was added with #692, which became part of If you can manage to do the changes before March 9th, then we can merge it before it becomes four years after the feature was added 😊 Thank you for contributing to documentation of Sinon 💯 |
@mroderick Challenge accepted! :) |
...for all past releases
For the record, the trick was:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! This was pending for a long time. Thank you very much ❤️
I am working on trying to make docs fixes a bit easier "for the rest of us", @mroderick. I'll have something soon. |
Let's talk about that during our call tomorrow ... one of the topics I'd like to talk about is documentation :) |
Thank you :) |
Closes #1606
Purpose (TL;DR) - mandatory
Adds documentation for an undocumented feature.
How to verify - mandatory
docs/_releases/latest/spies.md
Checklist for author
npm run lint
passes