Results 33 comments of Owen Stephens

I recently encountered this issue too. Having taken a look, @BurntSushi was right - the cause is an internal buffer being filled, then flushed to the underlying TabWriter, which causes...

Thanks @BurntSushi! FYI to build from source locally I had to give a more-specific `csv` version in `Cargo.toml` to allow the update from `1.0.1` to `1.1.2` (without it, `csv` only...

Hi @fatkodima, The classes are defined in a test, and it isn't otherwise necessary to explicitly name the class (we are using RSpec and generating the subclass in a `let`...

Thanks @hainesr - apologies for missing the already-open PR. Your suggestion of enabling Zip64 when required sounds good

Hi @hainesr - just checking in on the progress of this? I've just been bitten again by this issue (my fault for not remembering to enable `write_zip64_support`) - it would...

Hi @MrMeemus / @andialbrecht - is there any chance of this PR being progressed? I've been digging into https://github.com/dbcli/pgcli/issues/954 (no completion in `pgcli` given a table named `events`) and I've...

Thanks for the report @snuyanzin. The example is a bit tricky, since I don't think C-style comments can be nested, so the query actually _does_ contain an unmatched `'` as...

Thanks for the link @snuyanzin - I didn't know about this feature of C-style comments in SQL! Having looked into it a bit more, the cause of this issue is...

You fixed the enter issue (#1114) - thanks! Just to record my findings for the other issue: it looks like the C-n/C-p issue is related to/caused by https://github.com/dbcli/pgcli/commit/19c3e0eeb81f4a231d0ebfd2c18c47215f85e572#diff-8c7ffac5f775dfaf49268b59aba8bd78R108 - if...

Having dug into this, `sqlparse` is (incorrectly, for Postgres) treating `events` as a keyword, rather than an identifier. I've followed up on a [promising-sounding PR](https://github.com/andialbrecht/sqlparse/pull/389#issuecomment-609014700) that would allow us to...