-
Notifications
You must be signed in to change notification settings - Fork 26.3k
Fix doc builds for bool kwargs #44686
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
Conversation
|
@mruberry @zasdfgbnm PTAL |
zasdfgbnm
left a comment
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.
Super cool. Thanks!
Codecov Report
@@ Coverage Diff @@
## master #44686 +/- ##
==========================================
- Coverage 67.95% 67.94% -0.01%
==========================================
Files 384 384
Lines 49743 49743
==========================================
- Hits 33801 33798 -3
- Misses 15942 15945 +3
Continue to review full report at Codecov.
|
mruberry
left a comment
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.
Thank you!
facebook-github-bot
left a comment
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.
@mruberry has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: Fixes #43669 The bool will still link to https://docs.python.org/3/library/functions.html#bool. Tested using bmm:  Pull Request resolved: #44686 Reviewed By: ngimel Differential Revision: D23703823 Pulled By: mruberry fbshipit-source-id: 7286afad084f5ab24a1254ad84e5d01907781c85
Fixes #43669
The bool will still link to https://docs.python.org/3/library/functions.html#bool.

Tested using bmm: