Peter Stenger

 😬 https://github.com/sqlfluff/sqlfluff/issues/6926 Apparently they have a github action that only checks changed files: https://github.com/sqlfluff/sqlfluff-github-actions/tree/main/menu_of_workflows/drizly

> Whoa, that's slow... If this is about CI speed, I could split apart the "Python" and "Other" checks. If about during dev, they have a SQLFluff language server for...

Let me know if I should pull the workflow split into another PR @nwalters512

- https://docs.sqlfluff.com/en/stable/reference/rules.html#rule-references.consistent - https://docs.sqlfluff.com/en/stable/reference/rules.html#rule-aliasing.unused - https://docs.sqlfluff.com/en/stable/reference/rules.html#rule-convention.join_condition - https://docs.sqlfluff.com/en/stable/reference/rules.html#rule-layout.select_modifiers - https://docs.sqlfluff.com/en/stable/reference/rules.html#rule-structure.unused_cte - https://docs.sqlfluff.com/en/stable/reference/rules.html#rule-structure.subquery - https://docs.sqlfluff.com/en/stable/reference/rules.html#rule-structure.join_condition_order - https://docs.sqlfluff.com/en/stable/reference/rules.html#rule-convention.statement_brackets

> `structure.unused_cte`: We have a lot of false positives for this, so we couldn't turn it on. We frequently use CTEs for side effects; the rule doesn't seem to know...

I think the UI is in a state where I would be happy merging as is.


In [1bdc282](https://github.com/PrairieLearn/PrairieLearn/pull/12211/commits/1bdc282488b3f174ca5ac23165a0474552d42236), I hide the final resize indicator if it won't work (i.e. tablewidth < visible width). I also removed the right padding on the table.

Marking @nwalters512 as the primary reviewer on this PR @miguelaenlle .

> Have you thought of using the Makefile with the original intended use of targets, namely creating a target `.venv/bin/python3`, then making `python-deps` depend on that target? I'm not sure...