-
Notifications
You must be signed in to change notification settings - Fork 9
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.
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
Enable Gherkin Linting #191
base: main
Are you sure you want to change the base?
Enable Gherkin Linting #191
Conversation
@rartych I do not know how to manage pending linter errors. |
It looks that angle brackets |
Thanks for the feedback and info Rafal. Yes I think we can move forward with proposed above solution |
@rartych, @bigludo7 I will wait some time in order to align with Commonalities/PR#292 when merged. I think it be considered for MetaRelease Spring 25. And in case not finally merged, just merge this as part of WG work for MetaRelease Spring 25 as it works. |
What type of PR is this?
What this PR does / why we need it:
PR to enable Gherkin Linter as per Commonalities/PR#292.
Even it is ongoing PR, it is fine to incorporate into WG to have some preliminary linter rules
Which issue(s) this PR fixes:
Fixes #180
Special notes for reviewers:
Changes with regards to Commonalities/PR#292
Indentation (NOTE: reviewed test_definitions in other WGs and this makes sense)
"Background" set to "2".
"example" set to "6".
Disable "file-name" kebab-case checking as
operationId
is lowerCamelCase.Specific for Carrier Billing: disable "validate" word in Feature as Carrier Billing has
validatePayment
operationChangelog input
Additional documentation
N/A