feat(rules): update operation-tag-defined rule
#1876
Conversation
🦋 Changeset detectedLatest commit: 4ef0862 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
jeremyfiel
force-pushed
the
feat/update-operation-tag-rule
branch
2 times, most recently
from
208a581 to
e8011cc
Compare
| "@redocly/cli": patch | ||
| --- | ||
|
|
||
| Updated `operation-tag-defined` configurable rule. |
There was a problem hiding this comment.
what was was updated about the rule and why?
There was a problem hiding this comment.
The rule didn't check for a tags definition at the operation level. I also added the tests for it.
There was a problem hiding this comment.
What was the issue you're trying to solve @jeremyfiel? Was the linter failing? Or just not reporting the problem?
There was a problem hiding this comment.
the rule was not reporting if operation tags are not defined, it only reports if operation tags are defined but the global tag is missing.
jeremyfiel
force-pushed
the
feat/update-operation-tag-rule
branch
from
e8011cc to
6f4484c
Compare
jeremyfiel
force-pushed
the
feat/update-operation-tag-rule
branch
from
6f4484c to
8b1a1ae
Compare
check for `tags` existence in `Operation` node`
jeremyfiel
force-pushed
the
feat/update-operation-tag-rule
branch
from
8b1a1ae to
4ef0862
Compare
| "@redocly/cli": patch | ||
| --- | ||
|
|
||
| Updated `operation-tag-defined` configurable rule. |
There was a problem hiding this comment.
| Updated `operation-tag-defined` configurable rule. | |
| Updated the `operation-tag-defined` built-in rule to check that the globally defined tags are used at the operation level. |
There was a problem hiding this comment.
I was only able to find the built-in rule operation-tag-defined. I also think it would be helpful to add some information to the rule's documentation with this update - https://redocly.com/docs/cli/rules/oas/operation-tag-defined
adds a check for
tagsexistence inOperationnodeWhat/Why/How?
Reference
Testing
Screenshots (optional)
Check yourself
Security