Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Double check editor integration #3

Open
1 task
zth opened this issue Feb 2, 2023 · 3 comments
Open
1 task

Double check editor integration #3

zth opened this issue Feb 2, 2023 · 3 comments

Comments

@zth
Copy link
Collaborator

zth commented Feb 2, 2023

Make sure that the editor experience using Core is as good as possible.

  • Ensure that hovers make sense. E.g. we see no Core__Array.t<'typ>, but always array<'typ>, for builtins.
@cknitt
Copy link
Member

cknitt commented Feb 4, 2023

It would be quite useful if this could be a general thing for aliases to basic types like string, array etc.

E.g. always show string instead of Js.String.t, Js.String2.t, Belt.String.t, Belt.Map.String.key etc.

@zth
Copy link
Collaborator Author

zth commented Feb 4, 2023

Yes, for sure, I agree

@zth
Copy link
Collaborator Author

zth commented Feb 10, 2023

Module level docstrings would be nice here too. Ie if you hover a module, you currently don't see the module level documentation.

@zth zth mentioned this issue Feb 10, 2023
49 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants