Gears comments

Results 138 comments of


                                            Gears

Convert variable names to snake_case

All done now. - I've made the various refactors you suggested - The `check_valid_name` functions have now been reworked into two functions: `check_name_case` and `correct_name_case` - `check_name_case` returns an error...

LSP Code Action: Convert `let assert Ok` into a case expression

I've made some changes: - This code action now applies to any pattern - There didn't seem to be a `visit_pattern` method in the ast visitor, nor seemingly is the...

LSP Code Action: Convert `let assert Ok` into a case expression

I have now added the pattern visiting methods to the AST visitor trait

LSP Code Action: Convert `let assert Ok` into a case expression

With the case of no variables, I somehow overlooked a mistake in my implementation. That should now be fixed.

LSP Code Action: Convert `let assert Ok` into a case expression

The tests failing on macos seems to be a network issue?

LSP Code Action: Convert `let assert Ok` into a case expression

I've now fixed that. The code action applies to anywhere between the `let` keyword and just before the value, as both those locations were already stored in the AST. The...

Allow `gleam add` to take a version specifier

I think #3316 should have closed this

LSP Code Action: Add missing patterns to inexhaustive case expression

I've updated the parts which you suggested and are directly related to this PR. I guess I'll leave this as a draft for now, and open an issue to improve...

LSP Code Action: Add missing patterns to inexhaustive case expression

Alright, that case should be fixed now

LSP Code Action: Add missing patterns to inexhaustive case expression

Actually there's one more case I want to fix before this is merged: ```gleam let value = case a {} ``` Since it determines the indentation based on the start...