Morgante Pell

Results 282 comments of Morgante Pell

I think there's two immediate things we could do: - Adjust [this logic](https://github.com/getgrit/gritql/blob/44e1536af813f62c5c48dedc781aed56e3089cc9/crates/cli/src/analyze.rs#L208-L218) so that if the path of the file is directly specified (vs. being discovered via traversal) we...

Adding `.pyi` seems reasonable, as well as fixing the overall problem.

I think formatting itself is out of scope for GritQL, but this is a scenario we could likely support in our upcoming workflow engine + SDK. The syntax would look...

Awesome to see this coming together! I'm not sure I like using rewrites for the diagnostics: - While the quick-fix itself is available in the "fixer" section it seems somewhat...

> Does this mean we want to guarantee user has biome already installed and grit doctor will check that prerequisite? No, like with other formatters we should just skip formatting...

Code looks goodish but tests aren't passing - you probably need to update the GitHub actions to install Biome.

I don't mind clicking approve, but you should also be able to run tests locally. It looks like some snapshots will need updating, and the standard library tests will also...