You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In this PR, oad reports many instances of "AddingRequiredParameter" that are false positives, because the parameter was present and required in the previous version, but was specified at the path level rather than in the operation, as it is in the new version.
While this is a change in the way the API is described, it is not actually a change in the API definition, so should not be reported as a breaking change.
The text was updated successfully, but these errors were encountered:
I think I may see what is going on. The PR was updated but the PR checks were not marked as "outdated" or updated in place. This makes it look like the current code has problems when in fact the problems were fixed.
If this is correct then we can close this issue but should open one on the pipeline to update the PR checks section in place.
In this PR, oad reports many instances of "AddingRequiredParameter" that are false positives, because the parameter was present and required in the previous version, but was specified at the path level rather than in the operation, as it is in the new version.
While this is a change in the way the API is described, it is not actually a change in the API definition, so should not be reported as a breaking change.
The text was updated successfully, but these errors were encountered: