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

feat: rename /read-only to /read #1743

Closed
wants to merge 1 commit into from
Closed

Conversation

fry69
Copy link
Contributor

@fry69 fry69 commented Sep 26, 2024

fix #1739

Since /read and /read-only gets used interchangeably I think it is time to rename /read-only to only /read to reduce confusion.

Rationale:
/file clearly tells the user that a file should be marked to get edited
/read clearly communicates that the file should only be read, not edited

This is a one-time thing to learn and removes the mental load from often getting auto-corrected to/read-only.

IMHO it is enough to point this out and explain this in the documentation. aider should be as helpful and concise on the command line as possible.

Kinda reverts commit f9471fc

@fry69 fry69 mentioned this pull request Sep 26, 2024
@fry69 fry69 force-pushed the read-only-fix branch 3 times, most recently from c0ed3f7 to 6a445c0 Compare September 27, 2024 04:29
@paul-gauthier
Copy link
Collaborator

What's the purpose here? You can type /read foo.ext in the chat already.

@fry69
Copy link
Contributor Author

fry69 commented Sep 27, 2024

The purpose is to get rid of the read-only confusion (it's enough to point that out in the docs IMHO) and not get auto-completed to that, it is a bit annoying and inconsistent.

Feel free to close this you think otherwise though :)

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.

/read command is missing
2 participants