Skip to content

feat: use openapi.json by default#872

Closed
fourcels wants to merge 2 commits intodanielgtaylor:mainfrom
fourcels:feat-docs
Closed

feat: use openapi.json by default#872
fourcels wants to merge 2 commits intodanielgtaylor:mainfrom
fourcels:feat-docs

Conversation

@fourcels
Copy link
Copy Markdown
Contributor

openapi.json can directly view in browser

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.56%. Comparing base (4fd3cdb) to head (e8e7784).
⚠️ Report is 33 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #872   +/-   ##
=======================================
  Coverage   92.56%   92.56%           
=======================================
  Files          23       23           
  Lines        5511     5511           
=======================================
  Hits         5101     5101           
  Misses        351      351           
  Partials       59       59           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@danielgtaylor
Copy link
Copy Markdown
Owner

@fourcels why do you want this change? The doc generator supports YAML just fine.

@danielgtaylor danielgtaylor added the question Further information is requested label Nov 14, 2025
@wolveix
Copy link
Copy Markdown
Collaborator

wolveix commented Feb 16, 2026

Hey @fourcels! I'm going to close this as I'm not sure there's any reason to switch from YAML as the default, particularly considering that both are served anyway and YAML is the standard for OpenAPI specifications.

@wolveix wolveix closed this Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants