Skip to content

Feat: allow formatting specific models via path posargs for more typi…#3331

Merged
georgesittas merged 1 commit intoSQLMesh:mainfrom
z3z1ma:feat/format-specific-models
Nov 6, 2024
Merged

Feat: allow formatting specific models via path posargs for more typi…#3331
georgesittas merged 1 commit intoSQLMesh:mainfrom
z3z1ma:feat/format-specific-models

Conversation

@z3z1ma
Copy link
Contributor

@z3z1ma z3z1ma commented Nov 5, 2024

…cal and robust cli behavior.

This allows sqlmesh format to be used much more prescriptively. Furthermore it can be used in pre-commit hooks correctly this way too. The all or nothing was very limiting so we get this discernment almost for free given small change.

@z3z1ma
Copy link
Contributor Author

z3z1ma commented Nov 5, 2024

I think a diagnostic print of print(f"Formatting {target._path}", file=sys.stderr) would make the execution less obscure to the end user too but I'll let you weigh in.

@z3z1ma z3z1ma force-pushed the feat/format-specific-models branch from 9acb72c to b1db150 Compare November 5, 2024 21:45
@z3z1ma z3z1ma force-pushed the feat/format-specific-models branch from b1db150 to ecd1f2f Compare November 5, 2024 23:47
@georgesittas georgesittas merged commit b3adc72 into SQLMesh:main Nov 6, 2024
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.

3 participants