Domizio Demichelis

Results 134 comments of Domizio Demichelis

> you can use docker-compose.files setting? No I cannot because `docker-compose.files` uses only the filename, not the full path. Unless you have a small project or a big one very...

In Ubuntu 18.04 it is the same. However the Compose Up and Compose Down contextual menus work on the composer file, but it looks like the Extension Explorer is not...

Just discovered the Compose Up and Down menus are provided by the Docker extension, not the Docker Compose extension.

For test, it would probably be a good idea to use AR and the `/pagy/test/files/db/test.sqlite3` as is for this feature. BTW, the tests are currently just failing the coverage, so...

Sorry for the late answer... The test of the `:count_over` is done here: https://github.com/ddnexus/pagy/blob/e278b7020705f9e44a0f10c33eb56bdaf30db4ef/test/pagy/toolbox/paginators/offset_test.rb#L46-L58 As you see we mock it. 🤷🏻‍♂️ I have no idea about how to unify that.

@n-studio, the expert in async queries is @julianrubisch here... https://www.rorvswild.com/blog/2025/optimize-pagination-speed-with-asynchronous-queries-in-ruby-on-rails

> We should delay the call to value to the very end, ideally when it's time to render the pagination view. It makes sense to me, but... @julianrubisch ? If...

@n-studio the count integer is necessary to create an offset paginator instance. If your observation about calling value immediately is right, then the only way to get the async advantage,...

> .value is called asynchronously at any point and ONLY called synchronously if the main thread "surpassed" the spawned query executor thread. Rails put some nifty safeguard there. @julianrubisch I...

> @ddnexus I see... I don't think count is necessary for the offset, right? Count is necessary for the Pagy::Offset instantiation.