Skip to content

Conversation

@Tenebrae311
Copy link
Contributor

@Tenebrae311 Tenebrae311 commented Nov 1, 2025

  • added 3 licenses (MIT for source code, CC-BY-4.0 for docs and similar files, CC0-1.0 for template code without copyright)
  • licensed all current files in the repo
  • added lint command to the makefile
  • added lint-licensing stage to the ci

Emil Badura added 2 commits November 1, 2025 15:45
@Tenebrae311 Tenebrae311 linked an issue Nov 1, 2025 that may be closed by this pull request
Emil Badura added 3 commits November 1, 2025 15:54
Signed-off-by: Emil Badura <[email protected]>
Signed-off-by: Emil Badura <[email protected]>
Signed-off-by: Emil Badura <[email protected]>
Copy link
Collaborator

@fhilgers fhilgers left a comment

Choose a reason for hiding this comment

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

@Tenebrae311 Looks good 👍, one minor nit: the year appears twice in the headers, e.g. SPDX-FileCopyrightText: 2025 2025 robot-visual-perception

You can fix that in this PR or in a follow up PR, if you want this to already be merged, that's up to you

Emil Badura added 5 commits November 3, 2025 12:26
Signed-off-by: Emil Badura <[email protected]>
Signed-off-by: Emil Badura <[email protected]>
fix
Signed-off-by: Emil Badura <[email protected]>
Signed-off-by: Emil Badura <[email protected]>
Signed-off-by: Emil Badura <[email protected]>
@Tenebrae311
Copy link
Contributor Author

should be fixed now @fhilgers

Copy link
Collaborator

@fhilgers fhilgers left a comment

Choose a reason for hiding this comment

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

Thank You, LGTM 👍

@Tenebrae311 Tenebrae311 merged commit 8e992ac into main Nov 3, 2025
9 checks passed
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.

Setup Licensing

3 participants