Skip to content

Conversation

@williamjamir
Copy link
Contributor

The humanize package does not currently include the LICEN[CS]E file when uploaded to PyPI, causing tools like pip-license to fail to detect that this package is MIT licensed.

By updating the reference to LICENCE, we ensure that the correct file is included in the package.

@hugovk hugovk added the changelog: Fixed For any bug fixes label Mar 24, 2025
@codecov
Copy link

codecov bot commented Mar 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.49%. Comparing base (da7f7f1) to head (8eb0d24).
Report is 15 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #243   +/-   ##
=======================================
  Coverage   99.49%   99.49%           
=======================================
  Files          11       11           
  Lines         796      796           
=======================================
  Hits          792      792           
  Misses          4        4           
Flag Coverage Δ
macos-latest 97.73% <ø> (ø)
ubuntu-latest 97.73% <ø> (ø)
windows-latest 95.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hugovk
Copy link
Member

hugovk commented Mar 24, 2025

Thanks for the fix, the test failures are unrelated.

@hugovk hugovk enabled auto-merge (squash) March 24, 2025 14:53
@hugovk hugovk merged commit 0802447 into python-humanize:main Mar 24, 2025
34 checks passed
@hugovk hugovk changed the title Update license file reference from LICENSE to LICENCE in pyproject.toml Update license file reference from LICENSE to LICENCE in pyproject.toml Mar 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: Fixed For any bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants