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

Delete redundant setup.cfg files #6122

Merged
merged 4 commits into from
Jun 23, 2023

Conversation

pavoljuhas
Copy link
Collaborator

According to the setuptools docs, license_file is a deprecated keyword
replaced by the license_files keyword which defaults to "LICENSE" anyway.

Ref: https://github.com/pypa/setuptools/blob/v67.8.0/docs/references/keywords.rst?plain=1#L154-L163

According to the setuptools docs, `license_file` is a deprecated keyword
replaced by the `license_files` keyword which defaults to "LICENSE" anyway.

Ref: https://github.com/pypa/setuptools/blob/v67.8.0/docs/references/keywords.rst?plain=1#L154-L163
@CirqBot CirqBot added the size: S 10< lines changed <50 label Jun 6, 2023
@pavoljuhas pavoljuhas marked this pull request as ready for review June 21, 2023 01:03
@pavoljuhas pavoljuhas requested a review from tanujkhattar June 23, 2023 21:39
@tanujkhattar tanujkhattar enabled auto-merge (squash) June 23, 2023 21:42
@tanujkhattar tanujkhattar merged commit dfedcd6 into quantumlib:master Jun 23, 2023
@pavoljuhas pavoljuhas deleted the drop-setup.cfg branch June 24, 2023 00:16
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
According to the setuptools docs, `license_file` is a deprecated keyword
replaced by the `license_files` keyword which defaults to "LICENSE" anyway.

Ref: https://github.com/pypa/setuptools/blob/v67.8.0/docs/references/keywords.rst?plain=1#L154-L163

Co-authored-by: Tanuj Khattar <tanujkhattar@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: S 10< lines changed <50
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants