anicca icon indicating copy to clipboard operation
anicca copied to clipboard

`paths.changed.{path}` does not need the `{ operations_added, operations_changed, operations_removed }` layer

Open gr2m opened this issue 4 years ago • 2 comments
trafficstars

Example:

image

E.g. instead of

paths.changed.paths["/app"].operations_changed.get

you could just do

paths.change.paths["/app"].get

The operations_changed is implied by the changed

gr2m avatar Aug 31 '21 00:08 gr2m

Oh I think I get it, there might be a case when an operation is removed in a path, in which case we would get

paths.changed.paths["/app"].operations_removed.get

but I don't think that this is currently implemented that way?

gr2m avatar Aug 31 '21 00:08 gr2m

I think it would be easier to grasp if you all three of paths.{added,changed,removed} had the same format as paths in the source/target OpenAPI files (paths[path][method]), that way it's clear to see which are the operations that are added, which have been changed, which have been removed.

gr2m avatar Aug 31 '21 01:08 gr2m