Skip to content

Linting rules update 01 2026#564

Merged
rartych merged 7 commits intocamaraproject:mainfrom
rartych:linting-update-12_2025
Jan 19, 2026
Merged

Linting rules update 01 2026#564
rartych merged 7 commits intocamaraproject:mainfrom
rartych:linting-update-12_2025

Conversation

@rartych
Copy link
Copy Markdown
Contributor

@rartych rartych commented Dec 8, 2025

What type of PR is this?

  • correction

What this PR does / why we need it:

Fixes in gherkin_lint partly implemented in tooling:

  • Update max-scenarios-per-file configuration
  • Increase name length limits in linting rules

Spectral rules

  • Deprecate camara-discriminator-use rule
  • Revise Linting Rules Documentation

Which issue(s) this PR fixes:

Fixes #520

Does this PR introduce a breaking change?

  • Yes
  • No

Special notes for reviewers:

Final resolution on linting rules location should be done within #482

Changelog input

Linting rules updated

Additional documentation

Change countOutlineExamples setting from true to false.
`name-length` extended to 250 chars
`scenario-size` switched off
'camara-discriminator-use' rule deprecated
@rartych rartych changed the title Linting rules update 12 2025 Linting rules update 01 2026 Jan 12, 2026
@rartych rartych marked this pull request as ready for review January 12, 2026 08:44
patrice-conil
patrice-conil previously approved these changes Jan 12, 2026
Copy link
Copy Markdown
Contributor

@patrice-conil patrice-conil left a comment

Choose a reason for hiding this comment

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

LGTM

# - 09.01.2025: Updated info-contact rule
# - 21.07.2025: Added camara-schema-type-check rule
# - 05.12.2025: camara-discriminator-use depracated
# - 12.01.2026: camara-discriminator-use depracated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

depracated -> deprecated

Copy link
Copy Markdown
Collaborator

@Kevsy Kevsy left a comment

Choose a reason for hiding this comment

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

typo (see above)

PedroDiez
PedroDiez previously approved these changes Jan 12, 2026
Copy link
Copy Markdown
Contributor

@PedroDiez PedroDiez left a comment

Choose a reason for hiding this comment

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

LGTM

(with point identified by Kevin)

@rartych rartych dismissed stale reviews from PedroDiez and patrice-conil via 0d60ca9 January 12, 2026 13:31
@rartych rartych requested a review from Kevsy January 12, 2026 13:33
@rartych
Copy link
Copy Markdown
Contributor Author

rartych commented Jan 12, 2026

typo (see above)

Thanks, fixed in 0d60ca9

Copy link
Copy Markdown
Contributor

@PedroDiez PedroDiez left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@patrice-conil patrice-conil left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@Kevsy Kevsy left a comment

Choose a reason for hiding this comment

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

LGTM

@rartych rartych merged commit 7cb02e1 into camaraproject:main Jan 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proposal for management of step name length and number of steps within centralized linter model

4 participants