Skip to content

Conversation

eakmanrq
Copy link
Contributor

Prior to this change we considered SQLMesh versions "compatible", or that they could be run together, if they shared a common sqlglot minor version and were on the same migration release. Sometimes though we introduce incompatibility that doesn't require a migration and we represent this with a minor version bump. Therefore this change now considers SQLMesh's minor version too when deciding if releases are compatible.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This looks like a typo in this test. error was defined but never used. Updated to check for it similar to what is done below.

@eakmanrq eakmanrq force-pushed the eakmanrq/add_sqlmesh_version_check branch from f63ef8d to 4ed192a Compare February 14, 2024 20:21
@eakmanrq eakmanrq merged commit c602f8b into main Feb 15, 2024
@eakmanrq eakmanrq deleted the eakmanrq/add_sqlmesh_version_check branch February 15, 2024 19:01
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