Skip to content

Feat/openapi-sync-ci-check#709

Merged
Ahmedhossamdev merged 10 commits intomainfrom
feat/openapi-sync-ci-check
Mar 14, 2026
Merged

Feat/openapi-sync-ci-check#709
Ahmedhossamdev merged 10 commits intomainfrom
feat/openapi-sync-ci-check

Conversation

@Ahmedhossamdev
Copy link
Member

fix: #706

go tool sqlc generate -f gtfsdb/sqlc.yml

# Fetch the latest upstream OpenAPI spec and overwrite testdata/openapi.yml.
update-openapi:
Copy link
Member

Choose a reason for hiding this comment

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

After this PR gets merged, please extract this into a separate bash script.

Copy link
Member Author

Choose a reason for hiding this comment

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

Cool!

# Check whether testdata/openapi.yml matches the live upstream (skipping header).
# Exits 1 if out of date (useful for CI checks).
check-openapi:
@echo "Checking upstream OpenAPI spec for changes..."
Copy link
Member

Choose a reason for hiding this comment

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

Same here

Copy link
Member

@aaronbrethorst aaronbrethorst left a comment

Choose a reason for hiding this comment

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

Lgtm!

@Ahmedhossamdev Ahmedhossamdev merged commit 31adb88 into main Mar 14, 2026
5 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.

feat: detect stale OpenAPI spec and fail CI if out of sync with sdk-config

2 participants