Add optional BigQuery dry-run validation for article SQL examples#10
Merged
Conversation
- scripts/test-queries.js: extracts <!-- sqltest -->-decorated ```sql blocks from src/content/articles/ and validates them against live BigQuery schemas via free dry-run queries (npm run test:queries). Skips gracefully without gcloud credentials. --all tests undecorated blocks; <!-- sqltest: skip --> excludes a block permanently. - .github/workflows/query-tests.yml: runs decorated-query validation on PRs touching articles (requires GCP_SA_KEY secret; skips on forks).
Collaborator
Author
|
In order for me to test this PR I'll need to have a GCP_SA_KEY created as follows: Then provide me with the text in key.json so I can set it as a variable in this GitHub repo. |
Replace the GCP_SA_KEY secret with keyless OIDC auth against the measurement-lab kb-query-tests service account. Skip the job on fork PRs, which cannot mint OIDC tokens.
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
1 participant
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.
This PR creates a dry-run test of BigQuery SQL syntax in knowledgebase articles.
<!-- sqltest -->-decorated sql blocks from src/content/articles/ and validates them against live BigQuery schemas via free dry-run queries (npm run test:queries). Skips gracefully without gcloud credentials.<!-- sqltest: skip -->excludes a block.