Skip to content

chore: skip typing_extensions license check#9120

Merged
anakin87 merged 4 commits intomainfrom
typing-extensions-license-compliance
Mar 27, 2025
Merged

chore: skip typing_extensions license check#9120
anakin87 merged 4 commits intomainfrom
typing-extensions-license-compliance

Conversation

@anakin87
Copy link
Copy Markdown
Member

Related Issues

Proposed Changes:

  • add typing_extensions to excluded packages

How did you test it?

Tested locally using the docker image:

docker run -it -v `pwd`:/volume \
    --rm pilosus/pip-license-checker \
    java -jar app.jar --exclude 'typing_extensions.*' \
    --requirements '/volume/requirements.txt' \
    --verbose

Checklist

  • I have read the contributors guidelines and the code of conduct
  • I have updated the related issue with new insights and changes
  • I added unit tests and updated the docstrings
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I documented my code
  • I ran pre-commit hooks and fixed any issue

@anakin87 anakin87 marked this pull request as ready for review March 27, 2025 10:26
@anakin87 anakin87 requested a review from a team as a code owner March 27, 2025 10:26
@anakin87 anakin87 requested review from julian-risch and removed request for a team March 27, 2025 10:26
@anakin87 anakin87 added the ignore-for-release-notes PRs with this flag won't be included in the release notes. label Mar 27, 2025
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Mar 27, 2025

Pull Request Test Coverage Report for Build 14105778459

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 10 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.001%) to 90.166%

Files with Coverage Reduction New Missed Lines %
utils/hf.py 10 85.45%
Totals Coverage Status
Change from base Build 14105772409: -0.001%
Covered Lines: 9976
Relevant Lines: 11064

💛 - Coveralls

@anakin87 anakin87 enabled auto-merge (squash) March 27, 2025 11:08
@anakin87 anakin87 merged commit 06512a8 into main Mar 27, 2025
19 checks passed
@anakin87 anakin87 deleted the typing-extensions-license-compliance branch March 27, 2025 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release-notes PRs with this flag won't be included in the release notes. topic:CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants