Skip to content
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

Base.Bottom is referred to in the Union{} doc string, but it's not public #55292

Closed
nsajko opened this issue Jul 28, 2024 · 7 comments
Closed
Labels
docs This change adds or pertains to documentation

Comments

@nsajko
Copy link
Contributor

nsajko commented Jul 28, 2024

https://docs.julialang.org/en/v1/base/base/#Union{}

Usage of objects named "Bottom" among packages: https://juliahub.com/ui/Search?type=symbols&q=Bottom&u=use

Solutions, two alternatives:

  1. make Base.Bottom public
  2. delete Bottom from the Base and from the doc string of Union{}. This could be a viable minor change, after a PkgEval, etc.
nsajko added a commit to nsajko/julia that referenced this issue Jul 28, 2024
Having it is unnecessary, given `Union{}`, and confusing, now that
`Top` is gone.

Fixes JuliaLang#55292
@Keno
Copy link
Member

Keno commented Jul 28, 2024

I don't see why this is a problem.

@nsajko
Copy link
Contributor Author

nsajko commented Jul 28, 2024

It's a problem because Base.Bottom being referred to in the docs misleads new users to think that Bottom is public API. So I guess we want to either make it public, or delete it from the doc string?

@Keno
Copy link
Member

Keno commented Jul 28, 2024

It's a problem because Base.Bottom being referred to in the docs misleads new users to think that Bottom is public API

Why is this implied. Lots of doc strings talk about internals. We very explicitly moved away from documented == public, because it just leads to things not being documented.

@nsajko
Copy link
Contributor Author

nsajko commented Jul 28, 2024

OK then I was mistaken.

We very explicitly moved away from documented == public

Ref?

@LilithHafner
Copy link
Member

LilithHafner commented Aug 5, 2024

@nsajko nsajko added the docs This change adds or pertains to documentation label Aug 5, 2024
@nsajko
Copy link
Contributor Author

nsajko commented Aug 5, 2024

OK, then this is only a possible documentation style issue. I'd prefer the docs (except the devdocs, of course) to only document public interfaces, to be more newbie-friendly? Probably not worth it keeping the issue open.

@nsajko nsajko closed this as completed Aug 5, 2024
@LilithHafner
Copy link
Member

I think the solution for doc style is option ii in JuliaDocs/Documenter.jl#1506 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants