Skip to content

Add docstring for err in the REPL #48300

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.

Sign up for GitHub

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

Jump to bottom
Merged
KristofferC merged 2 commits into from
Jan 18, 2023
Merged

Add docstring for err in the REPL #48300

KristofferC merged 2 commits into from
Jan 18, 2023

Conversation

@Seelengrab
Copy link
Contributor

@Seelengrab Seelengrab commented Jan 16, 2023

Unlike ans, this previously wasn't documented:

help?> ans
search: ans transpose transcode contains expanduser instances MathConstants readlines LinearIndices leading_ones

  ans

  A variable referring to the last computed value, automatically set at the interactive prompt.

help?> err
search: error errormonitor ErrorException @error stderr InterruptException showerror LoadError InitError

Couldn't find err
Perhaps you meant error, zero, repr, eof, eps, esc, exp, end, nor, rm, xor, Ptr, for, try, @error, stderr or merge
  No documentation found.

  Binding err does not exist.

Found while looking at #48299

@jakobnissen
Copy link
Member

jakobnissen commented Jan 16, 2023

Is this documented in the Julia documentation? Should it be?

@ViralBShah ViralBShah added the docs This change adds or pertains to documentation label Jan 16, 2023
@Seelengrab
Copy link
Contributor Author

Seelengrab commented Jan 16, 2023 •
edited
Loading

It was originally added in #40642, which also mentioned it in NEWS.md. I think the docstring just got overlooked.

aviatesk
aviatesk approved these changes Jan 18, 2023
View reviewed changes
Copy link
Member

@aviatesk aviatesk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's include this in the Julia documentation also.

@aviatesk aviatesk added the merge me PR is reviewed. Merge when all tests are passing label Jan 18, 2023
@Seelengrab Seelengrab mentioned this pull request Jan 18, 2023
Closed
@KristofferC KristofferC merged commit 4e76346 into JuliaLang:master Jan 18, 2023
@DilumAluthge DilumAluthge removed the merge me PR is reviewed. Merge when all tests are passing label Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aviatesk aviatesk aviatesk approved these changes

Assignees

No one assigned

Labels

docs This change adds or pertains to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@Seelengrab @jakobnissen @aviatesk @ViralBShah @KristofferC @DilumAluthge