Skip to content

verify Swagger docs are up to date and fix annotations - #112

Merged
andrewtavis merged 1 commit into
scribe-org:mainfrom
ashb155:ci/swagger-docs
Oct 19, 2026
Merged

andrewtavis merged 1 commit into
scribe-org:mainfrom
ashb155:ci/swagger-docs

Conversation

@ashb155

@ashb155 ashb155 commented Oct 18, 2026

Copy link
Copy Markdown
Collaborator

Contributor checklist

  • This pull request is on a separate branch and not the main branch
  • I have ran the ./pre-commit executable as well as make lint and have fixed all reported issues

Description

This PR adds a CI step in .github/workflows/pr_ci.yaml to ensure generated Swagger documentation in docs/ remains up to date, and resolves the Swagger annotation errors across the codebase so the check passes out of the box.

Related issue

@andrewtavis

Copy link
Copy Markdown
Member

Great to have this, @ashb155! Thanks for the work here :)

@DeleMike DeleMike left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ashb155 , this looks great!

Thanks a ton!

@andrewtavis andrewtavis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From me as well, this looks great @ashb155!

@andrewtavis

Copy link
Copy Markdown
Member

Thanks so much for your contributions here 😊

@andrewtavis
andrewtavis merged commit 263ce13 into scribe-org:main Oct 19, 2026
4 checks passed
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.

Add a CI check that generated Swagger docs are up to date

3 participants