Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI action to ensure schema conformity on incoming PRs #7

Merged
merged 5 commits into from
Jan 21, 2025

Conversation

nullstalgia
Copy link
Member

@nullstalgia nullstalgia commented Jan 20, 2025

This PR adds an extra action that runs for pull requests and for pushes to develop. Using the flatbuffers compiler's "conformity check", we can automatically ensure we're not running afoul of the Schema Evolution rules specified here.

Only the schemas listed in .github/workflows/verify-list are checked.

Since this implementation of the verification pass relies on master branch being an authoritative source, care should be taken to avoid pushing directly to master without performing these same checks.

@nullstalgia nullstalgia marked this pull request as ready for review January 20, 2025 13:06
@hhvrc hhvrc merged commit af7b4f8 into master Jan 21, 2025
2 checks passed
@hhvrc hhvrc deleted the feature/ci-conformity-check branch January 21, 2025 00:44
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