Skip to content

fix: add openapi API routes#11

Merged
alexlovelltroy merged 1 commit into
mainfrom
fix/add-openapi-endpoints
Apr 25, 2026
Merged

fix: add openapi API routes#11
alexlovelltroy merged 1 commit into
mainfrom
fix/add-openapi-endpoints

Conversation

@synackd
Copy link
Copy Markdown
Contributor

@synackd synackd commented Apr 24, 2026

Pull Request Template

Thank you for your contribution! Please ensure the following before submitting:

Checklist

  • My code follows the style guidelines of this project
  • I have added/updated comments where needed
  • I have added tests that prove my fix is effective or my feature works
  • I have run make test (or equivalent) locally and all tests pass
  • DCO Sign-off: All commits are signed off (git commit -s) with my real name and email
  • REUSE Compliance:
    • Each new/modified source file has SPDX copyright and license headers
    • Any non-commentable files include a <filename>.license sidecar
    • All referenced licenses are present in the LICENSES/ directory

Description

Re-add missing OpenAPI endpoints:

  • /openapi.json: OpenAPI spec in JSON format
  • /docs: OpenAPI UI

Fabrica does not regenerate main.go, so these must be added manually.

To test:

  1. make build
  2. go run ./cmd/server/ serve
  3. curl localhost:8080/openapi.json
  4. curl localhost:8080/docs

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

For more info, see Contributing Guidelines.

- /openapi.json  - OpenAPI spec in JSON format
- /docs          - OpenAPI UI

Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
@synackd
Copy link
Copy Markdown
Contributor Author

synackd commented Apr 24, 2026

Related to OpenCHAMI/roadmap#124

@alexlovelltroy alexlovelltroy merged commit ac88a2f into main Apr 25, 2026
5 checks passed
@synackd synackd deleted the fix/add-openapi-endpoints branch April 27, 2026 02:25
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