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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add license #3

Merged
merged 6 commits into from
Apr 20, 2024
Merged

Add license #3

merged 6 commits into from
Apr 20, 2024

Conversation

stefanscherzinger
Copy link
Member

@stefanscherzinger stefanscherzinger commented Apr 19, 2024 •
edited
Loading

Steps

  • Check licenses per ament test in CMake
  • Add SPDX specifiers in the package.xml files
  • Add license notices to all development files
  • Add a top-level LICENSE file
  • Add a LICENSES folder with license notices
  • Add the MIT license and copyright for the json.hpp part
  • Add a license badge to the top-level readme

@stefanscherzinger stefanscherzinger force-pushed the add-license branch 2 times, most recently from 0faa685 to db0290e Compare April 19, 2024 14:37
@stefanscherzinger
Fix license notices
3dc0836 
They now comply with ament's copyright check.
@stefanscherzinger stefanscherzinger merged commit bd02b94 into main Apr 20, 2024
8 checks passed
@stefanscherzinger stefanscherzinger deleted the add-license branch April 20, 2024 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@stefanscherzinger