Skip to content

fix: rename stepLength to stepCount#20

Merged
k1LoW merged 1 commit intomainfrom
rename-step-count
Sep 10, 2025
Merged

fix: rename stepLength to stepCount#20
k1LoW merged 1 commit intomainfrom
rename-step-count

Conversation

@k1LoW
Copy link
Copy Markdown
Contributor

@k1LoW k1LoW commented Sep 10, 2025

This pull request renames the pipeline linting rule from "StepLength" to "StepCount" throughout the codebase and documentation to better reflect its purpose (limiting the number of steps in a pipeline). The changes ensure consistency across configuration files, code, tests, and documentation.

Renaming and Refactoring Pipeline Step Rule:

  • Renamed the struct StepLength to StepCount and updated its usage in the config/config.go file and related configuration logic. [1] [2]
  • Updated all references in the codebase from StepLength to StepCount, including variable names, configuration fields, and logic in tailor/lint.go.
  • Adjusted tests and test helpers to use StepCount instead of StepLength, ensuring tests reflect the new naming. [1] [2]

Documentation Updates:

  • Updated the documentation in README.md to replace "stepLength" with "stepCount" and revised descriptions to clarify that the rule limits the number of steps, not their length. [1] [2]

@k1LoW k1LoW self-assigned this Sep 10, 2025
@k1LoW k1LoW requested a review from a team as a code owner September 10, 2025 01:43
@github-actions
Copy link
Copy Markdown
Contributor

Code Metrics Report

Coverage Code to Test Ratio
32.5% 1:1.7

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

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

Reported by octocov

@k1LoW k1LoW merged commit 9770998 into main Sep 10, 2025
1 check passed
@k1LoW k1LoW deleted the rename-step-count branch September 10, 2025 02:32
@github-actions github-actions Bot mentioned this pull request Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants