Closed guicho271828 closed 5 years ago
<3 The Roswell support, the PR is a little on the fat side. I'll review it over the week.
I should clean up the commit a little bit.
I first separate the bugfix PR.
but hey, I have time recently, I can help cleaning up the other PRs too
want to make this repo in a more healthy state
Cleaned up the commits, hope I did nothing wrong during rebase.
I've reviewed this pull and found that links https://github.com/redline6561/coleslaw should be changed to https://github.com/kingcons/coleslaw.
Other staff is great!
By the way, @guicho271828 what do you think about a https://github.com/40ants/defmain? This could be a good replacement for trivia in terms of command-line arguments parsing.
It is up to you, I believe you can choose to switch later.
I wish other, smaller PRs get merged first
Actually, I'm not a repository's owner. Just decided to review interesting pull, because I've wrote a roswell script too, but it is much simpler than yours. I think it would be useful to have a command line utility out of the box.
I made the minimal changes since the the original 2-years old commits:
coleslaw-cli
package/system),Also, minor changes:
:swank
feature is found):clack
dynamically (which will cause problem in quicklisp)Both cli and repl commands have the unit test.
I guess it is fine now.
Command line syntax: