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

Make WHEEL parsing error line numbers one indexed. #2151

Merged
merged 1 commit into from
Mar 4, 2024

Conversation

konstin
Copy link
Member

@konstin konstin commented Mar 4, 2024

Fixes an off-by-one from #2149. enumerate is zero-based, human line numbers are one based.

Fixes an off-by-one from #2149. `enumerate` is zero-based, human line numbers are one based.
@konstin konstin added the error messages Messaging when something goes wrong label Mar 4, 2024
@konstin konstin enabled auto-merge (squash) March 4, 2024 11:50
@konstin konstin merged commit bc0345a into main Mar 4, 2024
7 checks passed
@konstin konstin deleted the konsti/wheel-error-is-off-by-one branch March 4, 2024 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
error messages Messaging when something goes wrong
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant