Skip to content

feat: add configurable acceptable warning threshold#7

Merged
k1LoW merged 1 commit into
mainfrom
acceptable
Sep 8, 2025
Merged

feat: add configurable acceptable warning threshold#7
k1LoW merged 1 commit into
mainfrom
acceptable

Conversation

@k1LoW
Copy link
Copy Markdown
Contributor

@k1LoW k1LoW commented Sep 8, 2025

This pull request refactors how lint configuration is structured and introduces a new feature to control the acceptable number of lint warnings before failing. The main changes are the introduction of an acceptable threshold for warnings, the grouping of all lint rules under a new rules field, and updates to both the code and documentation to reflect these changes.

Configuration structure improvements:

  • Added an acceptable field to the lint configuration, allowing users to specify the maximum number of lint warnings before the command fails. This is documented in the README.md and implemented in the lint command logic. [1] [2] [3] [4]
  • Refactored the configuration so that all lint rules are now grouped under a new rules field within the lint section, replacing the previous flat structure. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

Code and test updates:

Documentation:

  • Updated the README.md to document the new acceptable option and the new structure for specifying lint rules. [1] [2]

Other:

  • Minor code cleanup: removed an unused import from cmd/lint.go.

@k1LoW k1LoW self-assigned this Sep 8, 2025
@k1LoW k1LoW added enhancement New feature or request minor breaking change labels Sep 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 8, 2025

Code Metrics Report

Coverage Code to Test Ratio
33.0% 1:1.5

Code coverage of files in pull request scope (59.0%)

Files Coverage
cmd/lint.go 0.0%
config/config.go 0.0%
tailor/lint.go 98.4%

Reported by octocov

@k1LoW k1LoW merged commit 09b0b4d into main Sep 8, 2025
1 check passed
@k1LoW k1LoW deleted the acceptable branch September 8, 2025 10:36
@github-actions github-actions Bot mentioned this pull request Sep 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant