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

Remove select-get from Language Reference Docs #1747

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tbrinkman3
Copy link

Already documented in the Book. Removing duplicate location. See: #1744

@tbrinkman3 tbrinkman3 changed the title Remove select get Remove select-get from Language Reference Docs Jan 13, 2025
@fdncred
Copy link
Collaborator

fdncred commented Jan 13, 2025

I don't think it's a good idea to remove sections from the language guide because they're documented in the book. The language guide performs a slightly different task than the book and things like this should be minimally documented in the guide.

@NotTheDr01ds
Copy link
Contributor

NotTheDr01ds commented Jan 13, 2025

@fdncred To be fair, I added that page and TODO in the Language Guide a while back, then added the select/get info to the Book but forgot to remove it from the Language Guide. If there's something specific we need in the language guide regarding this, then I'm okay with leaving the placeholder, but it's my "fault" it's in there in the first place, so I'm okay with it coming out if you are.

@fdncred
Copy link
Collaborator

fdncred commented Jan 13, 2025

I think the language guide should every language related item in it. Even if it's just a high-level blurb that points to more in-depth documentation.

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.

3 participants