8000 Exclude comments from maximum line length checks by Witiko · Pull Request #43 · Witiko/expltools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Exclude comments from maximum line length checks #43

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
Merged
merged 8 commits into from
Feb 7, 2025
Merged

Exclude comments from maximum line length checks #43

merged 8 commits into from
Feb 7, 2025

Conversation

Witiko
Copy link
Owner
@Witiko Witiko commented Feb 6, 2025 •
edited
Loading

This PR makes the following changes:

  • Exclude comments from maximum line length checks. This includes spaces before the comments.

This PR also makes the following changes:

  • Correctly determine end column number for errorformat items %e and %k.

Closes #27.

@Witiko Witiko added the bug Something isn't working label Feb 6, 2025
@Witiko Witiko self-assigned this Feb 6, 2025
@Witiko Witiko mentioned this pull request Feb 7, 2025
Merged
@Witiko Witiko merged commit 011ce02 into main Feb 7, 2025
5 checks passed
@Witiko Witiko deleted the fix/s103-on-long-comment-lines branch February 7, 2025 08:54
@Witiko Witiko mentioned this pull request Feb 17, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Witiko Witiko

Labels
bug Something isn't working
Projects
None yet
Milestone
explcheck v0.7.0 (bug fixes, web app)
Development

Successfully merging this pull request may close these issues.

How to ignore S103 on too long comment lines
1 participant
@Witiko
0