open-interpreter icon indicating copy to clipboard operation
open-interpreter copied to clipboard

Published 20 hours ago • verified publisher icon OpenInterpreter

Fixed %tokens command

Open CyanideByte opened this issue 1 year ago • 1 comments

Describe the changes you have made:

Just updated variables to the new names to fix a crash

Reference any relevant issues (e.g. "Fixes #000"):

Pre-Submission Checklist (optional but appreciated):

  • [ ] I have included relevant documentation updates (stored in /docs)
  • [x] I have read docs/CONTRIBUTING.md
  • [x] I have read docs/ROADMAP.md

OS Tests (optional but appreciated):

  • [x] Tested on Windows
  • [ ] Tested on MacOS
  • [ ] Tested on Linux

CyanideByte avatar Jan 10 '24 06:01 CyanideByte

Nice catch! We should add all commands to tests @KillianLucas?

Notnaton avatar Jan 11 '24 17:01 Notnaton

Looks great! Many thanks @CyanideByte. I would have never caught this, and it totally breaks %tokens without your patch.

And I agree @Notnaton— though I'm not sure how to test %magic commands since they only exist for the terminal interface... I think we'll need to use a subprocess to run interpreter and test them (but they'll be hard to verify if they've passed...)

KillianLucas avatar Jan 18 '24 06:01 KillianLucas