Skip to content

refactor: openapi, add readme - #448

Merged
JAORMX merged 2 commits into
stacklok:mainfrom
ahmedrowaihi:openapi
May 22, 2025
Merged

JAORMX merged 2 commits into
stacklok:mainfrom
ahmedrowaihi:openapi

Conversation

@ahmedrowaihi

Copy link
Copy Markdown
Contributor

here is the docs :') , sorry for delay I thought I already included it
#399 (review)

@JAORMX
JAORMX merged commit 0da4f9a into stacklok:main May 22, 2025
@ahmedrowaihi
ahmedrowaihi deleted the openapi branch May 22, 2025 07:40
blkt pushed a commit to melbinjp/toolhive that referenced this pull request Aug 24, 2026
Three of them are in docs/server/README.md, which is itself in docs/server/
and names its own neighbours one directory up: swagger.json, swagger.yaml
and docs.go moved together in 0da4f9a (stacklok#448).

The other two name pkg/vmcp/server/telemetry.go, which became
pkg/vmcp/internal/backendtelemetry/backendtelemetry.go in 87fa2ac (stacklok#5457).

Git recorded all five as renames, so each destination is verified present
and each source verified absent at 0b8acf2. No prose changes.

Part of stacklok#6387, which lists ten more that need a judgement from inside the
project rather than a patch from outside.

Signed-off-by: Melbin J Paulose <melbinjpaulose@gmail.com>
blkt pushed a commit that referenced this pull request Aug 24, 2026
Three of them are in docs/server/README.md, which is itself in docs/server/
and names its own neighbours one directory up: swagger.json, swagger.yaml
and docs.go moved together in 0da4f9a (#448).

The other two name pkg/vmcp/server/telemetry.go, which became
pkg/vmcp/internal/backendtelemetry/backendtelemetry.go in 87fa2ac (#5457).

Git recorded all five as renames, so each destination is verified present
and each source verified absent at 0b8acf2. No prose changes.

Part of #6387, which lists ten more that need a judgement from inside the
project rather than a patch from outside.

Signed-off-by: Melbin J Paulose <melbinjpaulose@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants