Alan Cruickshank
Alan Cruickshank
I'm not planning to address the first issue here in the current reflow push because I think it makes indentation dependent on comma position: ```sql SELECT foo, -- single indent...
Ok - so I think the only additional feature we'd need to achieve this is allowing an optional additional _half indent_ (i.e. two spaces more than usual) for closing brackets...
Given the suggestion above of using `--nocolor` and the lack of other developments on this issue. I'm going to close it as stale.
I've already started merging in some of the logic for this, so I've assigned this issue to me for now.
@gwenwindflower - I think we've got a good solution for the Jinja parsing bit already 👍 The part that is causing me a headache is managing multiple versions of a...
@gwenwindflower - check out #5822 which supports a _very_ minimal version of this. It might be best to get on the phone to talk more though and work through some...
Thanks @tuzonghua - can you confirm whether this issue was present on 3.0.4 - or is it a new issue starting only in 3.0.5?
I got a bit confused for a bit, but I think I've found the problem. The example you've posted **doesn't** case the bug, but the log you attached shows why....
It looks like this might be something with python `multiprocessing` rather than sqlfluff directly: - https://scampsters.marcevanstein.com/t/leaked-semaphore-objects/329 - https://bugs.python.org/issue46391 Our use of `multiprocessing` under the hood is fairly limited. Using [Pool](https://python.readthedocs.io/en/stable/library/multiprocessing.html#multiprocessing.pool.Pool)...
Looks like this is now resolved upstream in cpython: https://github.com/python/cpython/issues/90549 Based on the python release schedule, I would expect (hope?) that this fix made it into python 3.11 (if not...