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

change to use non-deprecated header #1231

Merged
merged 1 commit into from
Apr 27, 2018

Conversation

wjwwood
Copy link
Member

@wjwwood wjwwood commented Apr 27, 2018

Looks like it was missed in this pr or has since regressed: #1217

@dhood
Copy link
Contributor

dhood commented Apr 27, 2018

Turns out it was missed by the auto-PR because of the # include format

@dhood dhood merged commit 4ff4dc5 into melodic-devel Apr 27, 2018
@dhood dhood deleted the fix_use_of_deprecated_header branch April 27, 2018 22:48
@dhood dhood removed the in progress label Apr 27, 2018
dhood pushed a commit that referenced this pull request Apr 27, 2018
@mikaelarguedas
Copy link
Contributor

Looks like it was missed in this pr

Yeah, as mentionned to @dhood offline, I didnt account for whitespaces when writing my search replace script :/. The last released version of pluginlib should not throw the warning anymore so there shouldnt be any rush releasing this

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