Skip to content
This repository has been archived by the owner on Sep 28, 2023. It is now read-only.

Updates license. #21

Merged
merged 2 commits into from
May 26, 2022
Merged

Updates license. #21

merged 2 commits into from
May 26, 2022

Conversation

francocipollone
Copy link
Collaborator

@francocipollone
Copy link
Collaborator Author

maliput_drake is a particular case. All the files were brought from Drake. Correspondent Drake license is already in the LICENSE file.

Do you consider that is there something else to do here? @agalbachicar @tfoote

@tfoote
Copy link
Collaborator

tfoote commented May 25, 2022

If it already has a license that should just persist.

agalbachicar
agalbachicar approved these changes May 26, 2022
View reviewed changes
Copy link
Collaborator

@agalbachicar agalbachicar left a comment

Choose a reason for hiding this comment

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

LGTM

@francocipollone francocipollone merged commit 38a48a1 into main May 26, 2022
@francocipollone francocipollone deleted the francocipollone/licensing branch May 26, 2022 13:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@agalbachicar agalbachicar agalbachicar approved these changes

@tfoote tfoote tfoote approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@francocipollone @tfoote @agalbachicar