-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement lingo init
for Zephyr and Native
#10
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really good. I guess u tested the init does the building work with the normal lfc ? And maybe to the quick cargo clippy
:D
Did you understand the clippy output? The |
I was just some problem with clap 3 I updated to clap 4 and the issue resolved itself. |
This panic there is that the CodeGenerator expects an absolute path to lfc and if the path doesn't exists we can only panic. |
But yeah regarding this fixme I would leave it for know and replaced with a search in $PATH with the help of this crate: https://crates.io/crates/which |
I think in order to merge this we should:
|
This PR implements the
lingo init
command for the Zephyr and the default (Native) platform. I went with the following command structure:lingo init --platform/-p zephyr --language/-l c
We use git to clone the
lf-west-template
down from Github and populate a project directory.