Gears comments

Results 138 comments of


                                            Gears

Favour public alias over internal definition for types in the language server

Currently, the language server prints the type alias if it's defined in the current module, or imported. However it doesn't yet have enough information to do it if it simply...

LSP: Suggestion on record accessors

I'm going to give this one a shot

Language Server drastically slows down on large codebase

This seems to be resolved by 4806e8d. It's still not super snappy, but much better than the experience demonstrated in that video.

support deprecated attribute in type constructors

Just so you know, merge commits are not allowed in this repository. You'll need to rebase before Louis will accept it

Confusing incorrect arity hint when labelled arguments are provided without labels

What should the hint say in this case? Just suggest the `c` label?

Fixed export package-definitions not including cached modules

From when I looked into implementing this, it seems the main limitation is type aliases. Type alias information isn't stored in the cached interface, so that would need to be...

Fixed export package-definitions not including cached modules

You don't need `UncompiledModule`. If you add type aliases to it, `ModuleInterface` should be enough

Fixed export package-definitions not including cached modules

I would say if you can reproduce the bug without your changes, then open a separate issue and let that be fixed separately. If this PR introduces that bug, then...

Fixed export package-definitions not including cached modules

Hi @mine-tech-oficial, are you still working on this PR? If not, I'm happy to pick up the work as it would be cool to get this fixed. Thanks!

Fixed export package-definitions not including cached modules

I believe this PR can be closed now as the related issue has now been fixed.