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

Feature/remove security directory #1

Closed
wants to merge 10 commits into from

Conversation

pablogs9
Copy link
Member

@pablogs9 pablogs9 commented Dec 3, 2019

No description provided.

jacobperron and others added 10 commits June 12, 2019 16:01
…os2#456)

If the maximum number is exceeded fail with an informative error message.

Fixes ros2#419.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: ivanpauno <ivanpauno@ekumenlabs.com>
* Increase MAX_STRING_SIZE

It's too short for string length.
It occurs the error when the string field in yaml files are too long....

Signed-off-by: Hyunseok Yang <hyunseok7.yang@lge.com>

* update test to match increased limit

Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
Signed-off-by: Zachary Michaels <zmichaels11@gmail.com>
…os2#501) (ros2#505)

* reset error message before setting a new one, embed the original one

Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>

* fix max line length

Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
Signed-off-by: Zachary Michaels <zmichaels11@gmail.com>
Signed-off-by: Michael Carroll <michael@openrobotics.org>
@pablogs9 pablogs9 closed this Dec 3, 2019
@Acuadros95 Acuadros95 deleted the feature/remove_security_directory branch June 6, 2023 13:08
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.

7 participants