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

Add general configuration for tests #53

Merged
merged 3 commits into from
Sep 21, 2020
Merged

Conversation

Zifzaf
Copy link
Contributor

@Zifzaf Zifzaf commented Sep 11, 2020

In an effort to move tests to be with the driver repo and not on there
own, the general configuration files for this setup are with the
embedded common repo. Add those configuration files.

@Zifzaf Zifzaf force-pushed the integrate_tests_to_driver_repo branch 2 times, most recently from cc258de to 18b7e16 Compare September 11, 2020 14:19
abrauchli
abrauchli previously approved these changes Sep 11, 2020
In an effort to move tests to be with the driver repo and not on there
own, the general configuration files for this setup are with the
embedded common repo. Add those configuration files.
Update readme to include a description of the new folder test-config.
@Zifzaf Zifzaf force-pushed the integrate_tests_to_driver_repo branch from d0c9097 to 94596a0 Compare September 16, 2020 08:39
Copy link
Contributor

@rnestler rnestler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should include the standard license header in each new source file as well.

test-config/sensirion_test_setup.cpp Show resolved Hide resolved
test-config/sensirion_test_setup.h Show resolved Hide resolved
Change test_setup to use integrated functions from sensirion_commen
@Zifzaf Zifzaf force-pushed the integrate_tests_to_driver_repo branch from 53d3347 to 5398d41 Compare September 21, 2020 11:52
@Zifzaf Zifzaf merged commit 285cb56 into master Sep 21, 2020
@rnestler rnestler deleted the integrate_tests_to_driver_repo branch September 21, 2020 15:36
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.

3 participants