-
-
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
Move docs inline from helpdb and add a couple doctests #22651
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,6 +18,12 @@ An array of the command line arguments passed to Julia, as strings. | |
""" | ||
const ARGS = String[] | ||
|
||
""" | ||
exit([code]) | ||
Quit (or control-D at the prompt). The default exit code is zero, indicating that the | ||
processes completed successfully. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. does processes being plural here sound redundant / a little off to anyone else? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sounded a little off on first read, yes. Technically more correct than a singular equivalent though I suppose. On that note, the "control-D at the prompt" snippet strikes me as out of place as well. Perhaps could be listed as a synonym instead? Also could use a ref to the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we should maybe note these as future todo cleanups somewhere There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Captured in an intro issue: #22765. Best! |
||
""" | ||
exit(n) = ccall(:jl_exit, Void, (Int32,), n) | ||
exit() = exit(0) | ||
quit() = exit() | ||
|
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.
(Tangent: This docstring for
full
only captures one of the various meanings offull
. Perhaps we should flesh out this docstring? Or not worry about it in anticipation of complete excision of full in this release cycle?)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.
If it were me I'd just leave it as-is, since that's a larger task than simply relocating the docstring.