Skip to content

SPDX: license declaration cleanup#68

Merged
neteler merged 2 commits intomainfrom
headers_update_SPDX_license_IDs
Nov 27, 2025
Merged

SPDX: license declaration cleanup#68
neteler merged 2 commits intomainfrom
headers_update_SPDX_license_IDs

Conversation

@neteler
Copy link
Member

@neteler neteler commented Nov 26, 2025

This PR fixes license declarations and adds missing declarations in file headers. It also adds the official license text file.

Goal: acceptable result with reuse lint

This PR fixes license declarations and adds missing declarations in file headers. It also adds the official license text file.

Goal: acceptable result with `reuse lint`
@neteler neteler self-assigned this Nov 26, 2025
@neteler neteler added the documentation Improvements or additions to documentation label Nov 26, 2025
Copy link
Contributor

Choose a reason for hiding this comment

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

[ruff] reported by reviewdog 🐶

module_pc_tpl, user_template = render_template(vp.module)

Copy link
Contributor

Choose a reason for hiding this comment

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

[ruff] reported by reviewdog 🐶

http_code, response_model = pickle.loads(self.response_data)

@neteler
Copy link
Member Author

neteler commented Nov 27, 2025

(the linting issues are unrelated to this PR)

Note: Some license confusion was added time ago in 556595c.

Copy link
Member

@mmacata mmacata 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! ✨

@neteler neteler merged commit 1ac4ab3 into main Nov 27, 2025
13 of 17 checks passed
@neteler neteler deleted the headers_update_SPDX_license_IDs branch November 27, 2025 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants