house-keeping - #26
Conversation
4fcf2eb to
77a5a68
Compare
|
TODOs:
|
csboo
left a comment
There was a problem hiding this comment.
Thanks for the work you put into this!
I mostly agree with the changes, good stuff..
I would remove the agents file as said before merging.
|
I'm not sure about the fish client either, i wouldn't suppose changing it to anything else though. Surely the best solution would be to write a proper seeder into the backend itself with a feature maybe. For now it is fine, but I'm not sure if it should be merged into main since it doesn't have much to do with the project. What do you think about this? @jarjk |
what's the "for now" in here? while the PR is in draft, or what? on the other hand, it enables you to use the project in it's current state, which means it does have something to do with it it's definitely useful to have some kind of a client to be able to set up the server properly before having an actual client I agree building an actual client in rust without #22 is painful and definitely not worth the time what do you say? I'll vibe a python client to see how bad it is though |
|
"for now" is until we find a better solution, and (as you yourself answered) that would involve many things like #22 ... |
its not the fish script that enables you to use the project, at the very least nothing stops you to run curls by hand and you dont even need fish for it, and its even cross platform
yeah but thats already wip, while there isnt an actual client that does that, this could stay in an issue and be just copy pasted there as a basic solution or sth
i dont think that would make much of a difference, who used windows anyways |
|
i see, that clears things up |
7858539 to
a9677da
Compare
fixes #5
adds readme, fish mock client, .env.example, very-very basic home-page
takes some weight off #4