Skip to content

Conversation

tylerjw
Copy link
Contributor

@tylerjw tylerjw commented Jul 29, 2022

Signed-off-by: Tyler Weaver [email protected]

This one requires more work to be integrated but I've created a library you can just include and use like this:

#include <parameter_validators/parameter_validators.hpp>

namespace pv = parameter_validators;

auto result = pv::contains<std::string>(param, {"a", "b"});

@tylerjw tylerjw force-pushed the parameter_validators branch from 95636ab to ef16221 Compare July 29, 2022 01:02
tylerjw added 3 commits July 28, 2022 21:14
Signed-off-by: Tyler Weaver <[email protected]>
Signed-off-by: Tyler Weaver <[email protected]>
@tylerjw tylerjw force-pushed the parameter_validators branch from b20b825 to abee36c Compare July 29, 2022 03:17
Copy link
Contributor

@pac48 pac48 left a comment

Choose a reason for hiding this comment

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

LGTM

@tylerjw tylerjw force-pushed the parameter_validators branch from 83f540f to 6dbe19d Compare July 29, 2022 03:40
@tylerjw tylerjw merged commit 79fcaca into PickNikRobotics:main Jul 29, 2022
@tylerjw tylerjw deleted the parameter_validators branch July 29, 2022 03:43
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