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

Docs: Reference functions instead of methods #38986

Merged
merged 3 commits into from
Apr 3, 2021

Conversation

BioTurboNick
Copy link
Contributor

@BioTurboNick BioTurboNick commented Dec 24, 2020

By referencing methods sometimes and not just functions, there was some duplication and confusion. For instance, we had get(::Any, ::Any, ::Any) method and also a get function, the latter of which duplicated the documentation for the former.

Is there an unseen benefit to referencing just the function at times? EDIT: Documenter errors if there are exact duplicates, and sometimes it makes sense to split the functions by section.

If this PR is acceptable, I can go through the other pages to do the same.

By referencing methods sometimes and not just functions, there was some duplication and confusion. For instance, we had `get(::Any, ::Any, ::Any)` method and also a `get` function, the latter of which duplicated the documentation for the former.

Is there an unseen benefit to referencing just the function at times?

If this PR is acceptable, I can go through the other pages to do the same.
@BioTurboNick BioTurboNick changed the title Reference functions instead of methods Docs: Reference functions instead of methods Dec 24, 2020
Copy link
Member

@fredrikekre fredrikekre left a comment

Choose a reason for hiding this comment

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

They are here for a reason; there are other places where methods of these functions are included. See CI failures.

@BioTurboNick
Copy link
Contributor Author

I think I made the only necessary change.

@ViralBShah
Copy link
Member

@fredrikekre Is this ok to merge?

@BioTurboNick
Copy link
Contributor Author

I narrowed the scope to just change it for get and get!, since there might be good reasons for the others.

@vtjnash vtjnash merged commit ad513ca into JuliaLang:master Apr 3, 2021
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
By referencing methods sometimes and not just functions, there was some duplication and confusion. For instance, we had `get(::Any, ::Any, ::Any)` method and also a `get` function, the latter of which duplicated the documentation for the former on the same page.
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
By referencing methods sometimes and not just functions, there was some duplication and confusion. For instance, we had `get(::Any, ::Any, ::Any)` method and also a `get` function, the latter of which duplicated the documentation for the former on the same page.
johanmon pushed a commit to johanmon/julia that referenced this pull request Jul 5, 2021
By referencing methods sometimes and not just functions, there was some duplication and confusion. For instance, we had `get(::Any, ::Any, ::Any)` method and also a `get` function, the latter of which duplicated the documentation for the former on the same page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants