IBM / openapi-validator

Configurable and extensible validator/linter for OpenAPI documents
Apache License 2.0
481 stars 88 forks source link

feat(new-rule): ibm-operation-summary-length #663

Closed dpopp07 closed 4 months ago

dpopp07 commented 4 months ago

PR summary

Adds a rule that verifies operation summaries are 80 characters or less in length, based on the summary writing guidance in the API Handbook.

A question for the PR reviewers. We already have these existing rules:

Should this check be merged into one of the existing rules or does it belong as its own rule? The granularity is nice for configuration but it might be confusing to have too many rules doing such similar things. What do y'all think?

PR Checklist

General checklist

Please make sure that your PR fulfills the following requirements:

Checklist for adding a new validation rule:

CLAassistant commented 4 months ago

CLA assistant check
All committers have signed the CLA.

ibm-devx-sdk commented 4 months ago

:tada: This PR is included in version 1.16.0 :tada:

The release is available on npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:

ibm-devx-sdk commented 4 months ago

:tada: This PR is included in version 1.17.0 :tada:

The release is available on npm package (@latest dist-tag)

Your semantic-release bot :package::rocket: