-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
RFC: Allow doctests in Markdown code #13045
Conversation
A good number were only blocked by having doctests.
1 | ||
0 | ||
0 | ||
```jl |
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.
this should also still be a doctest right?
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.
It wasn't originally. I believe the final two zeros are uninitialized, so it's not a deterministic test.
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.
right, nevermind
lgtm after removing the special case handling of |
RFC: Allow doctests in Markdown code
@@ -11666,22 +11593,18 @@ While it isn't normally necessary, user-defined types can override the default ` | |||
deepcopy | |||
|
|||
doc""" | |||
```rst | |||
.. widen(type | x) | |||
widen(type | x) |
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.
I'm getting the feeling this should be 2 separate docstrings
backported in #13072 |
Specify doctest with a ````jldoctest` code fence. I also converted the ~30 RST docstrings that were only blocked from conversion by a doctest code block.
CC @MichaelHatherly and @jakebolewski
Unfortunately this reflows the text within the generated stdlib, so the diff looks bigger than it actually is.