Enhancement: exclude_operations_with_extension support for specific values #2887
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi team, we use
opticfor breaking change detection, and it’s been very effective. However,exclude_operations_with_extensiondoesn’t fully support excluding unstable API endpoints for our use case. We extended its functionality in our fork and believe this enhancement could benefit other users, so we’d like to contribute it upstream.We have a custom
x-stability-levelextension that supports the following string valuesdraft,alpha,beta. We wanted to be able to configure what values indicate that the endpoint should be excluded from the breaking changes check, for example:🍗 Description
This PR introduces enhanced logic to the
excludeOperationWithExtensionMatchesutility inprojects/standard-rulesets/src/utils.ts. The function now supports excluding operations based on specific extension values, while previously it could only operate on "truthy" values.Extension exclusion logic now explicitly handles single extension strings, arrays of extension strings, and arrays of objects specifying extension keys and allowed values.