Ryan comments

Results 8 comments of


                                            Ryan

pre_visit_relation not invoked for Drop query

I just ran into this myself. I tried to do the naive thing and just add `#[cfg_attr(feature = "visitor", visit(with = "visit_relation"))]` as suggested by @ankrgyl, but that on its...

Question: why is the Visitor trait limited to statements, relations & expressions?

+1 to this – would be curious to try out your crate @freshtonic if/when it gets to a publishable state :)

[total-functions/no-unsafe-readonly-mutable-assignment] False positive?

I think I'm starting to wrap my head around this, but it's surprisingly complicated, so bear with me. In your example, the line `const destValue: Destination = sourceVal;` raises the...

@shopify/eslint-plugin: Add support for ESLint v9 and flat config

I'm curious what the status of this is or if there's any assistance that can be offered? I see that there's an open PR for the eslint-plugin-n migration (#388). Anywhere...

@shopify/eslint-plugin: Add support for ESLint v9 and flat config

Got it, thanks Ben! On Tue, Oct 1, 2024 at 5:31 PM Ben Scott ***@***.***> wrote: > eslint-plugin-import is still working on ESLint v9 support. > > There's nothing to...

typescript error TS2693: T only refers to a type, but is being used as a value here.

I'm running into this too. Did you ever find a solution?

Circular imports between `schemas` and `iso`

This seems to go all the way back to 4.0.0. Latest ^3.0.0 (3.25.76) is working for me.

minimumReleaseAge not working (shamefully-hoist)

I'm seeing similar behavior (though not in a workspace and not using shamefully-hoist). PNPM version 10.24.0, Node.js version 24.11.1.