search

Open-Source-Digital-Twin / furuta-pendulum-controller

MIT License
2 stars 1 forks source link

GitHub actions #12

Closed caiopiccirillo closed 1 year ago

caiopiccirillo commented 1 year ago

This changes will help us to check if the project compiles and if all unit tests are all OK. I also added a pipeline that check if the code is correctly formatted. And another one for checking typos.

caiopiccirillo commented 1 year ago

Please, don't merge before https://github.com/LLagu/HAL-controller-test/pull/11, because some pipelines will change due to dependency changes.

LLagu commented 1 year ago

Please, don't merge before #11, because some pipelines will change due to dependency changes.

Should be fine now right?

caiopiccirillo commented 1 year ago

As expected a lot of errors are popping up, because we injected the protobuf dependency. If you want to dive into this, let me know. If not, I'll fix this night 😄

LLagu commented 1 year ago

As expected a lot of errors are popping up, because we injected the protobuf dependency. If you want to dive into this, let me know. If not, I'll fix this night 😄

I see that it's mostly CMake related so I'll leave it to you for the moment :sweat_smile:. I'll take a look at your commits though!

caiopiccirillo commented 1 year ago

I tried something different here. I added support to vcpkg, which should make our lives easier regarding to dependencies. Locally the only change is that you'll need to rebuild your devcontainer. Initially I was planning to add windows as a supported platform, but my lazyness won this time 😆. So we'll support only linux x64 (and armhf/armv8 crosscompile). If you agree with the changes, you can merge.