search

ItJustWorksTM / SmartcarEmul

Apache License 2.0
4 stars 0 forks source link

Testing: Remove Urho #7

Closed RuthgerD closed 3 years ago

RuthgerD commented 3 years ago

urho is not nice :) Removes urho Removes vehicle config code Adds rapidjson stand alone instead of urhos one Removes urho from ci Fixes some cmake to be library friendly

platisd commented 3 years ago

If this is merged, will the project be functional/complete?

RuthgerD commented 3 years ago

If this is merged, will the project be functional/complete?

Yes because this PR merges into testing not master :) we intend to keep master as is until we have something better for the frontend (e.g. my Godot rewrite). The plan is to have that in a different repo though, so thats why I wanted to remove urho so that I can just focus on the library side of this project.

Anyways until then I intend to have testing as the mainline develop branch, hence the PR so that its visible

platisd commented 3 years ago

Oh I didn't notice this wasn't going towards master. Nevermind then. :eyes:

RuthgerD commented 3 years ago

Closing in favor of libSMCE