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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

馃殌 Add cmake macros for using tests with example yaml files. 馃 #57

Merged
merged 8 commits into from
Sep 6, 2022
Merged

馃殌 Add cmake macros for using tests with example yaml files. 馃 #57

merged 8 commits into from
Sep 6, 2022

Conversation

destogl
Copy link
Contributor

@destogl destogl commented Aug 31, 2022

  • Add macros for executing tests with provided yaml files.
  • Add documentation into readme.

@destogl destogl changed the title Add cmake macros for using tests with example yaml files ane the library. 馃殌 Add cmake macros for using tests with example yaml files ane the library. 馃 Aug 31, 2022
@destogl destogl changed the title 馃殌 Add cmake macros for using tests with example yaml files ane the library. 馃 馃殌 Add cmake macros for using tests with example yaml files. 馃 Aug 31, 2022
tylerjw
tylerjw reviewed Aug 31, 2022
View reviewed changes
example/CMakeLists.txt Outdated Show resolved Hide resolved
example/CMakeLists.txt Outdated Show resolved Hide resolved
generate_parameter_library/cmake/generate_parameter_library.cmake Outdated Show resolved Hide resolved
@destogl destogl requested a review from tylerjw September 3, 2022 10:21
tylerjw
tylerjw approved these changes Sep 6, 2022
View reviewed changes
Copy link
Contributor

@tylerjw tylerjw left a comment

Choose a reason for hiding this comment

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

This is a really useful cmake macro. Thank you for adding this. As the name changed, please update it in the readme.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@tylerjw tylerjw merged commit d3d78b9 into PickNikRobotics:main Sep 6, 2022
@destogl destogl deleted the add-cmake-macro-for-tests-with-yaml branch September 6, 2022 06:56
@destogl destogl mentioned this pull request Sep 6, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tylerjw tylerjw tylerjw approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@destogl @tylerjw