-
Notifications
You must be signed in to change notification settings - Fork 20
Update pyproject.toml license settings and minimum python version #139
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
Conversation
Miauwkeru
commented
Sep 19, 2025
- Change the minimum python version to 3.10
- Update deprecated license settings
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #139 +/- ##
==========================================
+ Coverage 92.98% 93.01% +0.03%
==========================================
Files 21 21
Lines 2437 2435 -2
==========================================
- Hits 2266 2265 -1
+ Misses 171 170 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
pyproject.toml
Outdated
| requires-python = "~=3.9" | ||
| license.text = "Apache License 2.0" | ||
| requires-python = ">=3.10" | ||
| license = "AGPL-3.0-or-later" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong license.
24bce5c to
67dc222
Compare
059c09f to
d14c986
Compare
Schamper
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update required ruff version.
d14c986 to
6d24f97
Compare
Schamper
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dissect/cstruct/types/base.py
63: # Python 3.9 compat thing for bound type vars