Gajus Kuizinas

Results 763 comments of Gajus Kuizinas
trafficstars

so this is how we had to patch safeql to make it work for Contra with Slonik. ```diff diff --git a/dist/index.mjs b/dist/index.mjs index 148393dd953dc663cbe65163523671e43cfe3baf..e172d5143a38ad5cb5f314078ba3d5508cc3e00c 100644 --- a/dist/index.mjs +++ b/dist/index.mjs @@...

> I don't think that it would be too much work and I'm looking for a contributor to help me out there. If you want to be the man it...

This appears to be a major issue. @anubra266 do you have a clue on what's the bottleneck?

Will spend some time debugging this today, but when I attempt to use this plugin in our project, ESLint goes entirely to a halt.

``` Rule | Time (ms) | Relative :----------------------------------|----------:|--------: @pandacss/file-not-included | 9434.809 | 14.8% @pandacss/no-unsafe-token-fn-usage | 5100.427 | 8.0% @pandacss/no-hardcoded-color | 4944.562 | 7.7% @pandacss/no-invalid-token-paths | 4529.854 | 7.1% @pandacss/no-dynamic-styling |...

I didn't validate this, but I am 80% sure that the reason for this is because the current implementation does not cache the results of: ``` const isFileIncluded = isValidFile(context)...

@anubra266 I will happily submit a PR if that's the blocker here.

Just flagging to the core team that this is a major issue that currently seems to be ignored. @anthonyshew @chris-olszewski

here is a complete patch that worked for us: ```Dockerfile # patch start -- https://github.com/vercel/turborepo/issues/9120#issuecomment-2580436623 RUN apt-get update && apt-get install -y jq RUN wget https://github.com/mikefarah/yq/releases/download/v4.40.5/yq_linux_amd64 -O /usr/bin/yq && chmod...