Skip to content

chore: move OpenAPI adjustments from Makefile to CI workflow#50

Open
sallymoc wants to merge 2 commits intomainfrom
chore/openapi-ci-adjustments
Open

chore: move OpenAPI adjustments from Makefile to CI workflow#50
sallymoc wants to merge 2 commits intomainfrom
chore/openapi-ci-adjustments

Conversation

@sallymoc
Copy link
Contributor

@sallymoc sallymoc commented Mar 5, 2026

Move yq post-processing (x-scalar-ignore on QubicLiveService) from the Makefile to the GitHub Actions workflow. The Makefile now generates raw protoc output, and adjustments are applied on-the-fly before dispatching the base64-encoded spec to consumer repos (integration, docs).

Move yq post-processing (x-scalar-ignore on QubicLiveService) from the
Makefile to the GitHub Actions workflow. The Makefile now generates raw
protoc output, and adjustments are applied on-the-fly before dispatching
the base64-encoded spec to consumer repos (integration, docs).
@sallymoc
Copy link
Contributor Author

sallymoc commented Mar 5, 2026

Related PRs (need to be merged before):
qubic/docs#62
qubic/integration#104

@sallymoc sallymoc force-pushed the chore/openapi-ci-adjustments branch from 02f1016 to e9bd05f Compare March 5, 2026 15:00
@sallymoc sallymoc requested review from 0xluk, LINCKODE and qubicmio March 5, 2026 15:02
Fails the PR if the committed OpenAPI YAML doesn't match what protoc
would generate from the current proto files.
@sallymoc sallymoc force-pushed the chore/openapi-ci-adjustments branch from e9bd05f to 1aff544 Compare March 5, 2026 15:03
- run: go test ./... No newline at end of file
- run: go test ./...

check-openapi:
Copy link
Member

@qubicmio qubicmio Mar 5, 2026

Choose a reason for hiding this comment

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

Not sure about this one because I don't know, if different plugin versions might generate slightly different output. (we can change this later, if we are not sure)

@qubicmio
Copy link
Member

qubicmio commented Mar 5, 2026

If it works (one question) then lgtm.

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