-
Notifications
You must be signed in to change notification settings - Fork 10k
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
Missing Matcher documentation #2463
Comments
Thank you for the issue! This has now been addressed internally and will be fixed in the next OSS push. (333288356) |
Please unsubscribe me from your mailing list
…On Wed, Sep 23, 2020, 8:06 PM CJ Johnson ***@***.***> wrote:
Thank you for the issue! This has now been addressed internally and will
be fixed in the next OSS push. (333288356)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#2463 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AO7RFJWWKLNPCYQHLXAV4Z3SHIBXDANCNFSM4IYDNGZQ>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In both the gMock and gtest documentations, a set of Matchers is described, and links are provided that suggest they should go to a complete listing of the Matchers. However, both just link back to the gMock cookbook itself.
The gMock link is located in the first line under the heading here: https://github.com/google/googletest/blob/master/googlemock/docs/cook_book.md#using-predicates-as-matchers
The gtest link can be seen here, described as a list of all matchers:
https://github.com/google/googletest/blob/master/googletest/docs/advanced.md#asserting-using-gmock-matchers
The bad link can be seen here: https://github.com/google/googletest/blob/master/googlemock/docs/cook_book.md#MatcherList
Would it be possible to update these links to either point to the correct location (if it exists), or delete them?
The text was updated successfully, but these errors were encountered: