Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨No code suggestions found for the PR. |
There was a problem hiding this comment.
Pull Request Overview
This PR adds two new Fortitude linter rule exclusions (C061 and OB011) to the ignore list in the configuration file. This appears to be a maintenance change to suppress additional linting warnings that were likely introduced by recent code changes or linter updates.
Key changes:
- Updated
.fortitude.tomlto ignore two additional linter rules
| @@ -1,4 +1,4 @@ | |||
| [check] | |||
| ignore = ["E001","S001","S101","C121","S091","MOD011","C001","S101","S102","C002","PORT011","C072","C003","C131","C141","C092"] | |||
| ignore = ["E001","S001","S101","C121","S091","MOD011","C001","S101","S102","C002","PORT011","C061","C072","C003","C131","C141","C092","OB011"] | |||
There was a problem hiding this comment.
The rule "S101" appears twice in the ignore list (positions 3 and 8). This duplication should be removed to keep the configuration clean and avoid confusion.
| ignore = ["E001","S001","S101","C121","S091","MOD011","C001","S101","S102","C002","PORT011","C061","C072","C003","C131","C141","C092","OB011"] | |
| ignore = ["E001","S001","S101","C121","S091","MOD011","C001","S102","C002","PORT011","C061","C072","C003","C131","C141","C092","OB011"] |
User description
fix lint
PR Type
Other
Description
File Walkthrough
.fortitude.toml
Update linter ignore rules.fortitude.toml
C061andOB011to the ignore list in the check configuration