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

cleanup: Use tox core "length" functions instead of constants. #174

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

iphydf
Copy link
Member

@iphydf iphydf commented Nov 30, 2024

These constants can change from version to version, so we should use a runtime version of them rather than encoding the number into our binary at compile time.


This change is Reviewable

@iphydf iphydf added this to the v1.18.0 milestone Nov 30, 2024
@github-actions github-actions bot added the cleanup Internal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature. label Nov 30, 2024
These constants can change from version to version, so we should use a
runtime version of them rather than encoding the number into our binary
at compile time.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Internal code cleanup, possibly affecting semantics, e.g. deleting a deprecated feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant