Skip to content

Set default extra-args to skip update checking and release notes#168

Closed
blahgeek wants to merge 1 commit intoMatthewZMD:mainfrom
blahgeek:yikai/default-args-no-update
Closed

Set default extra-args to skip update checking and release notes#168
blahgeek wants to merge 1 commit intoMatthewZMD:mainfrom
blahgeek:yikai/default-args-no-update

Conversation

@blahgeek
Copy link
Copy Markdown
Contributor

this one is a little bit opinionated.. personally I think it makes sense to skip these possible interruption during startup, to prevent aidermacs failing to send commands to the backend

@MatthewZMD
Copy link
Copy Markdown
Owner

Since Aider's LLM model capabilities is hard depended on aider's version, I think it's reasonable to remind user to update as much as possible. If we really want this feature, we can turn them into defcustoms. What do you think?

@blahgeek
Copy link
Copy Markdown
Contributor Author

I guess you have a point. If the user solely use aider in aidermacs, then it seems to be a bad idea to completely skip upgrade checking which would means that the user would never get upgrades.

My initial goal is to prevent the interruption of the startup, so that if the user implicitly start the backend by running some commands (e.g. "code change"), the backend would always be ready to accept the command, instead of losing the command input due to interruption. But now that I think about it, maybe it's not a good tradeoff given the low frequency of updates.

I think I would close this PR then. Thanks.

@blahgeek blahgeek closed this Aug 17, 2025
@MatthewZMD
Copy link
Copy Markdown
Owner

Thank you for all the contributions!

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.

2 participants