conjure icon indicating copy to clipboard operation
conjure copied to clipboard

Published 20 hours ago • verified publisher icon Olical

[WIP] Refactor and improve Common Lisp client

Open Dotrar opened this issue 2 years ago • 2 comments

This PR is to improve the Common Lisp client, mainly by refactoring the code and making it easier to read, and changing to the slynk client that is a more modern and maintained version of slime for common lisp.

TODO:

  • [ ] Confirm Slynk message encoding/decoding in remote.transport.slynk
  • [x] Refactor out utility functions into common-lisp.utils
  • [ ] Connect Common Lisp Client to slynk transport
  • [ ] Have helper / install script in wiki page ie: a one-liner to quickstart
  • [ ] Remove swank-y code.
  • [ ] General tidy up before merging.

Dotrar avatar Apr 27 '22 11:04 Dotrar

So what state would you say this is in right now? Still an improvement on what is there right now even if there's more you'd like to do? If you're kinda happy with how it behaves I'll be happy to merge and get it in the hands of users. Future changes are still of course welcome! 😄

Olical avatar May 28 '22 10:05 Olical

Oh actually this would be a breaking change right? Because it requires a different transport so the user would have to start their REPL differently?

Olical avatar May 28 '22 10:05 Olical

closing this PR because it's messy and out of date.

changing to slynk was a lot easier than i was expecting, check out my new PR here: #400

Dotrar avatar Aug 27 '22 09:08 Dotrar