Skip to content

Conversation

@jrhillery
Copy link
Contributor

Proposed change

The python -m script.licenses check was failing with msg:
Approved license detected for [email protected]: MIT -- None -- []
Please remove the package from the EXCEPTIONS list.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New integration (thank you!)
  • New feature (which adds functionality to an existing integration)
  • Deprecation (breaking change to happen in the future)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:
  • Link to documentation pull request:
  • Link to developer documentation pull request:
  • Link to frontend pull request:

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist
  • I have followed the perfect PR recommendations
  • The code has been formatted using Ruff (ruff format homeassistant tests)
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • The manifest file has all fields filled out correctly.
    Updated and included derived files by running: python3 -m script.hassfest.
  • New or updated dependencies have been added to requirements_all.txt.
    Updated by running python3 -m script.gen_requirements_all.
  • For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.

To help with the load of incoming pull requests:

@jrhillery jrhillery requested a review from a team as a code owner May 3, 2025 17:35
@home-assistant home-assistant bot added bugfix cla-signed small-pr PRs with less than 30 lines. labels May 3, 2025
@abmantis
Copy link
Member

abmantis commented May 3, 2025

Now it fails with the reverse error 🙃
Since we don't pin the setuptools version, it gets installed as a dependency from other packages and maybe the version will depend on the order?

I wonder if those packages should have setuptools as an install dependency. If so, should we pin setuptools version?

@bdraco
Copy link
Member

bdraco commented May 3, 2025

Now it fails with the reverse error 🙃 Since we don't pin the setuptools version, it gets installed as a dependency from other packages and maybe the version will depend on the order?

I wonder if those packages should have setuptools as an install dependency. If so, should we pin setuptools version?

Consider hardcoding an exception to ignore the setuptools license.
Since it's part of the Python standard packaging ecosystem, it's highly unlikely to pose any issues.

@jrhillery jrhillery marked this pull request as draft May 4, 2025 03:15
@thecode
Copy link
Member

thecode commented May 4, 2025

I didn't noticed the discussion here and merged #144181 but it seems to fix the problem (rebased branches after doesn't show any error)

@jrhillery jrhillery closed this May 4, 2025
@github-actions github-actions bot locked and limited conversation to collaborators May 5, 2025
@jrhillery jrhillery deleted the remove-setuptools-from-exceptions branch May 7, 2025 13:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bugfix cla-signed small-pr PRs with less than 30 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants