Skip to content

ci: extend sqlglot syntax check to mimic-iii concepts - #2144

Open
Chessing234 wants to merge 2 commits into
MIT-LCP:mainfrom
Chessing234:fix/ci-mimic-iii-sql-syntax-check
Open

ci: extend sqlglot syntax check to mimic-iii concepts#2144
Chessing234 wants to merge 2 commits into
MIT-LCP:mainfrom
Chessing234:fix/ci-mimic-iii-sql-syntax-check

Conversation

@Chessing234

Copy link
Copy Markdown
Contributor

Summary

  • Run check_sql_syntax.py on both mimic-iv/concepts and mimic-iii/concepts in CI
  • Trigger the workflow when mimic-iii concept SQL changes
  • Fix incorrect GCS test docstring copied from the bg test

Test plan

  • python .github/scripts/check_sql_syntax.py mimic-iii/concepts (109 files OK)
  • python .github/scripts/check_sql_syntax.py mimic-iv/concepts

Made with Cursor

Chessing234 and others added 2 commits July 31, 2026 13:16
Parse mimic-iii/concepts in PR checks alongside mimic-iv/concepts.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant