Skip to content

Adding LICENSE to Repository #132

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

Merged
merged 3 commits into from
Nov 30, 2022
Merged

Adding LICENSE to Repository #132

merged 3 commits into from
Nov 30, 2022

Conversation

chayim
Copy link
Contributor

@chayim chayim commented Oct 31, 2022

The README for this repository indicates that the code is BSD licensed, but no license is present. This commit adds the BSD license, back dating the copyright to the initial commit and author. Additionally, a README badge was added, for ease of use.

@illia-v
Copy link
Contributor

illia-v commented Nov 29, 2022

https://github.com/redis/hiredis-py/blob/master/COPYING is the license

@chayim
Copy link
Contributor Author

chayim commented Nov 29, 2022

Removed the dupe - but same license, just codified as LICENSE (note text).

@illia-v
Copy link
Contributor

illia-v commented Nov 29, 2022

You may need to update https://github.com/redis/hiredis-py/blob/master/MANIFEST.in too

@chayim
Copy link
Contributor Author

chayim commented Nov 30, 2022

I'll clean up packaging too in a future PR - after the modernizing one.

@chayim chayim merged commit b042bd9 into redis:master Nov 30, 2022
@chayim chayim deleted the ck-license branch November 30, 2022 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants