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.

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

Add license header to all python files. #139

Merged
merged 1 commit into from
Nov 14, 2018
Merged

Conversation

mzfr
Copy link
Contributor

@mzfr mzfr commented Nov 8, 2018

Fixes #134
I have added the small license header in all the python files in the tools.
Should I also add the headers to test files and test addon?

Also in the header will it be Copyright (C) 2018-2018 Team Kodi or Copyright (C) 2005-2018 Team Kodi?

@Rechi
Copy link
Member

Rechi commented Nov 8, 2018

Copyright (C) 2017-2018 Team Kodi as @razzeee started the tool in 2017.

@mzfr mzfr force-pushed the add-license-header branch from 3e50551 to de477f8 Compare November 9, 2018 05:38
@Rechi
Copy link
Member

Rechi commented Nov 10, 2018

You have to add a LICENSES/README.md file.

@mzfr mzfr force-pushed the add-license-header branch from de477f8 to d196aeb Compare November 11, 2018 15:16
Copy link
Member

@Rechi Rechi left a comment

Choose a reason for hiding this comment

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

LICENSES/GPL-3.0-only file is missing

LICENSES/BSD-3-Clause Outdated Show resolved Hide resolved
LICENSES/BSD-4-Clause Outdated Show resolved Hide resolved
LICENSES/GPL-2.0-only Outdated Show resolved Hide resolved
LICENSES/GPL-2.0-or-later Outdated Show resolved Hide resolved
LICENSES/LGPL-2.1-or-later Outdated Show resolved Hide resolved
LICENSES/README.md Outdated Show resolved Hide resolved
LICENSES/README.md Outdated Show resolved Hide resolved
LICENSES/README.md Outdated Show resolved Hide resolved
LICENSES/README.md Outdated Show resolved Hide resolved
LICENSES/Unlicense Outdated Show resolved Hide resolved
@mzfr mzfr force-pushed the add-license-header branch 3 times, most recently from 8f1cfa6 to f059890 Compare November 13, 2018 14:54
LICENSES/README.md Outdated Show resolved Hide resolved
LICENSES/README.md Outdated Show resolved Hide resolved
Also added the required LICENSES for reference.
@mzfr mzfr force-pushed the add-license-header branch from f059890 to 7d8ce22 Compare November 14, 2018 13:23
@Rechi Rechi merged commit 7909a12 into xbmc:master Nov 14, 2018
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.

2 participants