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

Unify PickNik name in copyrights #419

Merged
merged 1 commit into from
Apr 14, 2021
Merged

Conversation

tylerjw
Copy link
Member

@tylerjw tylerjw commented Apr 13, 2021

Description

Various copyrights mention PickNik Robotics (these are my fault), PickNik LLC (from moveit1, we should fix these there too), PickNik Consulting (only one of these). There is a legal requirement that these are a legal name. This fixes that.

@codecov
Copy link

codecov bot commented Apr 13, 2021

Codecov Report

Merging #419 (f39e0fa) into main (feaa27c) will increase coverage by 0.01%.
The diff coverage is n/a.

❗ Current head f39e0fa differs from pull request most recent head 8a9a639. Consider uploading reports for the commit 8a9a639 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main     #419      +/-   ##
==========================================
+ Coverage   52.97%   52.98%   +0.01%     
==========================================
  Files         212      212              
  Lines       22622    22622              
==========================================
+ Hits        11981    11983       +2     
+ Misses      10641    10639       -2     
Impacted Files Coverage Δ
...nclude/moveit/robot_state/cartesian_interpolator.h 85.72% <ø> (ø)
...it_core/robot_state/src/cartesian_interpolator.cpp 39.54% <ø> (ø)
moveit_ros/moveit_servo/src/servo_parameters.cpp 67.25% <ø> (ø)
...nning_scene_monitor/src/planning_scene_monitor.cpp 56.11% <0.00%> (-0.12%) ⬇️
...meterization/work_space/pose_model_state_space.cpp 83.02% <0.00%> (+1.89%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update feaa27c...8a9a639. Read the comment docs.

@tylerjw
Copy link
Member Author

tylerjw commented Apr 13, 2021

moveit_servo tests continue to be flaky, re-running jobs. We should probably fix the flakyness of those before trying to release for foxy or it will probably plague the official buildfarm.

@nbbrooks
Copy link
Contributor

nbbrooks commented Apr 13, 2021

Edited: I think we should use "PickNik Inc." and not "PickNik, Inc."

With this search
grep --ignore-case -r -P "PickNik(?! Inc.)" . | grep -v "picknik.ai"

I picked up a few things that should be resolved with
findreplace "PickNik, LLC" "PickNik Inc."
findreplace "PickNik nor" "PickNik Inc. nor"
findreplace "PickNik, Inc" "PickNik Inc"

Copy link
Contributor

@nbbrooks nbbrooks left a comment

Choose a reason for hiding this comment

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

lgtm thanks!

@tylerjw tylerjw merged commit cd6ec43 into moveit:main Apr 14, 2021
@tylerjw tylerjw deleted the rename-picknik branch April 14, 2021 17:19
@sjahr sjahr mentioned this pull request Apr 16, 2021
6 tasks
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.

None yet

3 participants