Don’t /drop read-only files added at launch
aider
Space Play/pause —
f Fullscreen —
←→ ±5s
Commentary
- 0:01 We’re going to update the /drop command to keep any read only files that were originally specified at launch.
- 0:10 We’ve added files that handle the main CLI and in-chat slash commands like /drop.
- 0:20 Let’s explain the needed change to aider.
- 1:20 Ok, let’s look at the code.
- 1:30 I’d prefer not to use “hasattr()”, let’s ask for improvements.
- 1:45 Let’s try some manual testing.
- 2:10 Looks good. Let’s check the existing test suite to ensure we didn’t break anything.
- 2:19 Let’s ask aider to add tests for this.
- 2:50 Tests look reasonable, we’re done!