Skip to content

chore: Add yamlfmt and ensure openapi spec stays formatted#248

Merged
blainekasten merged 4 commits intomainfrom
blaine/dx-505-add-json-linter-to-openapi-spec
Apr 23, 2026
Merged

chore: Add yamlfmt and ensure openapi spec stays formatted#248
blainekasten merged 4 commits intomainfrom
blaine/dx-505-add-json-linter-to-openapi-spec

Conversation

@blainekasten
Copy link
Copy Markdown
Contributor

@blainekasten blainekasten commented Apr 23, 2026

  • Adds yamlfmt to keep a common lint style on the openapi spec
  • Adds a workflow on PRs to ensure format has been applied
  • Runs the formatter on the spec

Comment thread .github/workflows/lint.yml Fixed
@blainekasten blainekasten force-pushed the blaine/dx-505-add-json-linter-to-openapi-spec branch from 6b67a4c to 3f9bcef Compare April 23, 2026 16:08
Comment thread .github/workflows/lint.yml Fixed
@blainekasten blainekasten force-pushed the blaine/dx-505-add-json-linter-to-openapi-spec branch from 3f9bcef to 444cc6f Compare April 23, 2026 16:43
blainekasten and others added 2 commits April 23, 2026 13:08
…ntain permissions'

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

✱ Stainless preview builds

This PR will update the togetherai SDKs with the following commit messages.

go

chore(internal): regenerate SDK with no functional changes

openapi

chore(internal): regenerate SDK with no functional changes

python

chore(internal): regenerate SDK with no functional changes

terraform

chore(internal): regenerate SDK with no functional changes

typescript

chore(internal): regenerate SDK with no functional changes
togetherai-openapi studio · code

Your SDK build had at least one "note" diagnostic.
generate ✅

⚠️ togetherai-python studio · code

Your SDK build had at least one "warning" diagnostic.
generate ⚠️build ⏭️lint ⏭️test ⏭️

⚠️ togetherai-go studio · code

Your SDK build had a failure in the test CI job, which is a regression from the base state.
generate ✅build ⏭️lint ✅test ❗

go get github.com/stainless-sdks/togetherai-go@ec96978a4c3a42a9e3b50e81a05fd83234c35714
⚠️ togetherai-typescript studio · code

Your SDK build had a failure in the test CI job, which is a regression from the base state.
generate ⚠️build ⏭️lint ⏭️test ❗

togetherai-terraform studio · code

Your SDK build had at least one "note" diagnostic.
generate ✅lint ✅test ✅


This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push.
If you push custom code to the preview branch, re-run this workflow to update the comment.
Last updated: 2026-04-23 22:07:39 UTC

@blainekasten blainekasten changed the title chore: Add yaml lint and ensure openapi spec stays formatted chore: Add yamlfmt and ensure openapi spec stays formatted Apr 23, 2026
@blainekasten
Copy link
Copy Markdown
Contributor Author

AI Review:

 Review: PR #248 — Add yamlfmt and ensure openapi spec stays formatted         
                                                                                
  Verdict: FORMATTING ONLY — no functional changes detected.                    
                                                                                
  What the PR does                                                              
                                                                                
  - Adds a .yamlfmt config file (2-space indent, LF line endings, retain line   
  breaks)                                                                       
  - Adds a GitHub Actions lint workflow that runs yamlfmt --dry-run to enforce  
  formatting on PRs and pushes to main                                          
  - Applies yamlfmt to openapi.yaml (the bulk of the diff: 813 lines changed)
  - Updates README.md with development instructions                             
                                                                                
  openapi.yaml analysis                                                         
                                                                                
  All 813 changed lines in openapi.yaml are purely cosmetic. No functional      
  differences found

@blainekasten blainekasten merged commit da83248 into main Apr 23, 2026
6 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.

3 participants