WilliamKarolDiCioccio / open_local_ui

OpenLocalUI: Native desktop app for Windows, MacOS and Linux. Easily run Large Language Models locally, no complex setups required. Inspired by OpenWebUI's simplicity for LLM use.
MIT License
16 stars 2 forks source link

Check in generated code to make building easier for starters #32

Closed Rossi1337 closed 3 weeks ago

Rossi1337 commented 3 weeks ago

Hi, I open this for further discussion. When I checkout now the code it is no longer easy to build it. Basically what I have to do today is run some different code generators and other tools before I can build the App.

To make things a little bit easier I would suggest to at least check in open_local_ui/services/protobufs/server.pbgrpc.dart

I do not have the protobuf compiler installed so I cannot generate this code easily. I assume this code will change only very rarely so I would suggest to check it in.

For the .env I do not have a good solution but I would suggest to check in a .env.test with the following content:

GITHUB_FEEDBACK_PAT= GITHUB_COLLABORATORS_PAT= SUPABASE_URL= SUPABASE_ANON_KEY=

So you can just copy that one and put it into .env to get started.

This would make checking out and building much easier.

WilliamKarolDiCioccio commented 3 weeks ago

Wilco! Taking care of it tomorrow. I was think right now I've even forgot to write it in the dev env setup instructions

WilliamKarolDiCioccio commented 3 weeks ago

Meanwhile i checked out the gRPC generated files. Tommorow I'll add development and testing environment variables and close the issue

WilliamKarolDiCioccio commented 3 weeks ago

@Rossi1337 are you satisfied with the update? We might also add a setup task to automatically generate all the code the first time. Hope it makes things easier also for new contributors

Rossi1337 commented 3 weeks ago

Yes it is super smooth now. At least if you have some basic experience with Flutter. Checkout code. Add the .env file. Then in pubspec.yaml click once "Get Packages" and your are done and ready to launch the App. I don't think a setup task is necessary.