Gears comments

Results 138 comments of


                                            Gears

Calling a record constructor without arguments is accept in const definition

Wouldn't this be a breaking change though?

Calling a record constructor without arguments is accept in const definition

Good point. Formatter to the rescue!

Sourcemaps

I don't think the author of that PR is planning to continue it, so nobody is working on this at the moment

Compiler crash after editing gleam.toml to use remote version of package, but not editing manifest.toml

This should be fixed by #5141, which will be part of the next Gleam release.

Record update with mixed positional and named fields drops positional fields

No, that doesn't seem to be what they're reporting here. I would expect the value to be `T(1, a: 4, b: 3)` after the update, and certainly `b`, which is...

Record update with mixed positional and named fields drops positional fields

I'll look into this

Record update with mixed positional and named fields drops positional fields

Should I wait for #5061 to be merged before fixing this?

feat: only consider local type variables in error's type printing

They should go in the `type_::tests::errors` module

LSP code action to create unknown module

As I mentioned in the PR, my vote is for allowing it on any module, as I feel that makes the most sense. Modules don't always import submodules so I...

LSP code action to create unknown module

I think undrafting the PR is fine at this stage. If anyone has more thoughts about this they can comment here or on the PR