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

set up watchdog to notify AutoCompleter when files are changed externally #1544

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

Conversation

jbellis
Copy link
Contributor

@jbellis jbellis commented Sep 15, 2024

Fixes #1533

@fry69
Copy link
Contributor

fry69 commented Sep 22, 2024

@jbellis Does pressing Enter work for you to refresh the autocompleter (as @paul-gauthier mentioned in the related issue) and is this an acceptable solution for you?

@jbellis
Copy link
Contributor Author

jbellis commented Sep 22, 2024

For me personally, sure. I thought this is the kind of quality-of-life improvement that would be nice to have as aider matures and broadens its userbase, but if I am wrong feel free to close it out.

@jbellis
Copy link
Contributor Author

jbellis commented Sep 26, 2024

Update: I find that taking a few seconds wondering why completion isn't working, then C-a C-k to preserve my partial command, enter, C-y to edit again, is distracting and I would definitely prefer to have it refresh automatically in the background.

@CLAassistant
Copy link

CLAassistant commented Oct 16, 2024

CLA assistant check
All committers have signed the CLA.

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.

Feature request: play nice with external editing
3 participants