Skip to content
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

feat: add elixir code generation #5

Merged
merged 2 commits into from
Dec 6, 2023

Conversation

noaccOS
Copy link
Collaborator

@noaccOS noaccOS commented Oct 9, 2023

add nix package and ci action for elixir code generation.
works the same way as the rust code generation.

depends on #4 (included here)

adds elixir code generation and ci tests.
there is also some minor refactor to #3
there is also a fix to the ci where it would check the diff after a make all, but the build directory is gitignored so it would always pass.

@noaccOS noaccOS marked this pull request as draft October 12, 2023 10:57
@noaccOS noaccOS force-pushed the feat/elixir-gen branch 7 times, most recently from 71d2e1b to 99b27eb Compare October 12, 2023 16:03
@noaccOS noaccOS changed the title feat(ci): add elixir code generation feat: add elixir code generation Oct 12, 2023
@noaccOS noaccOS marked this pull request as ready for review October 12, 2023 16:11
@noaccOS noaccOS force-pushed the feat/elixir-gen branch 2 times, most recently from b858885 to 92506fc Compare October 17, 2023 10:50
@noaccOS noaccOS marked this pull request as draft December 6, 2023 14:05
the file was not updated when we moved from `make all` to `make install`
for adding files in the libraries

Signed-off-by: Francesco Noacco <[email protected]>
@noaccOS noaccOS force-pushed the feat/elixir-gen branch 4 times, most recently from 01d24b6 to 1e73496 Compare December 6, 2023 16:08
Signed-off-by: Francesco Noacco <[email protected]>
@noaccOS noaccOS marked this pull request as ready for review December 6, 2023 16:15
@rbino rbino merged commit 3c25312 into edgehog-device-manager:main Dec 6, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants