Simplify uses of `resolve-refs` partial

zecakeh commented 3 months ago

Since it is a recursing partial, we can call it once we get the data, and it will solve everything in the tree. In the end only the http-api shortcode was not doing that already.
Enable a lint in the OpenAPI validator to only allow $ref where it is supposed to be. It depends on #1772 for the lint to pass.
Document our use of $ref inside examples to allow to compose them from other examples, although it is not supported by the OpenAPI and JSON Schema specs.
Fix a call to resolve-refs, even if it doesn't seem to actually be needed because calls to this shortcode point to schemas without $ref (which is why the bug was unnoticed).

zecakeh commented 2 months ago

Rebased on main to pass CI

richvdh commented 2 months ago

Rebased on main to pass CI

could you merge rather than rebase next time, please. Rebasing makes it hard to see what has changed since an earlier review.

matrix-org / matrix-spec