-
Notifications
You must be signed in to change notification settings - Fork 12
fix: Update output: printer to avoid range(a, b, 0) #138
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
If `names` has length <n you get a ValueError. I assume the 4 is there so the output isn't spammed. Maybe it should be configurable.
Thanks for identifying and proposing a fix for this. Once you rename the PR to follow conventional titles and sign the CLA, we can approve and merge. |
Haha, odd indeed. |
Caution This pull request contains changes to GitHub workflows! |
I'm not sure what the bot is doing here, but if you accept the CLA we can look at merging soon. |
I thought I had already, sorry. |
Thanks for the contribution. |
Description
If
names
has length <n you get a ValueError.Type of Change
Issue Number
I decided to push this instead of an issue.
Code Compatibility
Code Performance and Testing
Dependencies
Documentation
Additional Notes
I assume the 4 is there so the output isn't spammed. Maybe it should be configurable.