-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add environmental module #84
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.
Was it necessary to check in the generated code to git? I prefer to keep them in the build folder and not commit them in.
a0c1f51
to
271109f
Compare
Necessary is a strong word, but it is in line with how it's done in NCS. We can refactor this if needed. |
I think we should avoid checking in generated code, its not necessary. Its up to use to decide. For instance for wifi prov stuff I didn't do it. I think you also can do some tricks that prevents the code from being generated in source, but rather in build. That is supported when using protobuf. |
This is doable using cmake. I can help out if needed. |
b183cc9
to
79044a0
Compare
done |
64b440a
to
d2072f8
Compare
This patch adds a module to report gas sensor measurements using the BSEC library. Signed-off-by: Maximilian Deubel <maximilian.deubel@nordicsemi.no>
d2072f8
to
e8067fe
Compare
This patch changes the transport module's subscriber to a message subscriber. Signed-off-by: Maximilian Deubel <maximilian.deubel@nordicsemi.no>
e8067fe
to
b57ae82
Compare
Quality Gate passedIssues Measures |
No description provided.