Chris Papademetrious
Chris Papademetrious
@dvarrazzo - when I use the **asynchronous** code structure in this pull request (complete runnable example [here](https://github.com/sqlalchemy/sqlalchemy/discussions/12522#discussioncomment-13024666)) with logging enabled, I get the following warning: ``` venv/lib/python3.11/site-packages/psycopg_pool/pool_async.py:142: RuntimeWarning: opening the...
@dvarrazzo - thanks! I updated the asynchronous example [there](https://github.com/sqlalchemy/sqlalchemy/discussions/12522#discussioncomment-13024666) to open the pool outside the constructor.
@CaselIT, @dvarrazzo - I am fine with waiting for the psycogp_pool 3.3.0 release, then releasing this documentation update with the more user-friendly example (noting the required psycogp_pool version).
@dvarrazzo - for completeness, I will also keep both the < 3.3 and >= 3.3 asynchronous examples [here](https://github.com/sqlalchemy/sqlalchemy/discussions/12522#discussioncomment-13024666) as a reference.
This was fixed in `minify-html==0.16.0`.
@raducoravu, your proposal is also consistent with the HTML5 `` element itself, which defaults to being an inline element. @stefan-jung - for #4434, do you want me to propose a...
@raducoravu - I confirmed that #4537 will fix this issue: ```html Click the button. Click the button. ```
@jelovirt - is this difficult to fix? I am trying to work on #4488 and this makes it difficult to instrument the code and understand what is happening.
@dvarrazzo - I like the idea of enhancing psycogp_pool 3.3.0 to remove the need to subclass `psycopg.AsyncConnection`. The retroactive support for psycopg < 3.3 would be a nice bonus. Thanks...
@raducoravu - In the output HTML, I think the attribute group name should be used as the attribute name, like this: ```html KEEP (no attribute group) KEEP (attribute group) ```...