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

Align SPDX header for Python files #100

Merged
merged 1 commit into from
May 11, 2023

Conversation

erikbosch
Copy link
Contributor

@erikbosch erikbosch commented May 11, 2023

This is to make sure all our Python files fulfill our current SPDX check

This PR do:

  • Adds SPDX to all python files that did not had it before
  • Add our generic copyright to those files (without SPDX) that did not had it before. In some cases this means that explicit Bosch copyright is removed
  • Adds python3 shebang for all executables.
  • Also making all "top level" python files executable.

Adding copyright and SPDX to all *.py files
Copy link
Contributor

@SebastianSchildt SebastianSchildt left a comment

Choose a reason for hiding this comment

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

Not a fan of shebang with space. But "the Internet" says it is perfectly fine https://unix.stackexchange.com/questions/276751/is-space-allowed-between-and-bin-bash-in-shebang

lgtm 🦛

@SebastianSchildt SebastianSchildt merged commit b5c183f into eclipse-kuksa:main May 11, 2023
@erikbosch erikbosch deleted the erikbosch/erik_csv branch September 29, 2023 09:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants