[gha] ensure bundled openapi.yaml is in sync#243
Merged
Conversation
matthappens
approved these changes
Mar 3, 2026
62da724 to
7f85025
Compare
7f85025 to
db19163
Compare
AaryamanBhute
approved these changes
Mar 3, 2026
Contributor
Greptile SummaryThis PR adds a GitHub Actions workflow step to enforce that the bundled Key changes:
Confidence Score: 4/5
|
| - name: Check compiled OpenAPI is up to date | ||
| run: | | ||
| if ! git diff --exit-code openapi.yaml mintlify/openapi.yaml; then | ||
| echo "ERROR: Compiled openapi.yaml is out of sync with source files in openapi/." |
Contributor
There was a problem hiding this comment.
The error message only mentions openapi.yaml, but the git diff check on line 44 validates both openapi.yaml and mintlify/openapi.yaml. If only mintlify/openapi.yaml is out of sync, this message will be misleading.
Suggested change
| echo "ERROR: Compiled openapi.yaml is out of sync with source files in openapi/." | |
| echo "ERROR: Compiled openapi.yaml or mintlify/openapi.yaml is out of sync with source files in openapi/." |
Prompt To Fix With AI
This is a comment left during a code review.
Path: .github/workflows/openapi-build.yml
Line: 45
Comment:
The error message only mentions `openapi.yaml`, but the `git diff` check on line 44 validates both `openapi.yaml` and `mintlify/openapi.yaml`. If only `mintlify/openapi.yaml` is out of sync, this message will be misleading.
```suggestion
echo "ERROR: Compiled openapi.yaml or mintlify/openapi.yaml is out of sync with source files in openapi/."
```
How can I resolve this? If you propose a fix, please make it concise.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

No description provided.