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.

Sign up for GitHub

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

Jump to bottom

reduce pylint max line length to 80 from default of 100 #258

Merged
merged 1 commit into from
Jul 25, 2024
Merged

reduce pylint max line length to 80 from default of 100 #258

merged 1 commit into from
Jul 25, 2024

Conversation

spencerschrock
Copy link
Contributor

Summary

This matches the limit previously used with flake8 before it was replaced in #232.

Release Note

NONE

Documentation

NONE

@spencerschrock
reduce pylint max line length to 80 from default of 100
fc2b227 
This matches the limit previously used with flake8 before it was
replaced in 373cc53.

Signed-off-by: Spencer Schrock <sschrock@google.com>
@spencerschrock spencerschrock requested review from a team as code owners July 25, 2024 16:12
mihaimaruseac
mihaimaruseac approved these changes Jul 25, 2024
View reviewed changes
Copy link
Collaborator

@mihaimaruseac mihaimaruseac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. I didn't know the default was 100.

@mihaimaruseac mihaimaruseac merged commit f409c9a into sigstore:main Jul 25, 2024
20 checks passed
@spencerschrock spencerschrock deleted the restore-line-len branch July 25, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mihaimaruseac mihaimaruseac mihaimaruseac approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@spencerschrock @mihaimaruseac