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

feat: start recovering errors locally instead of at the top level #177

Merged
merged 5 commits into from
Jun 16, 2024

Conversation

aboeglin
Copy link
Contributor

@aboeglin aboeglin commented Jun 8, 2024

Goal is to keep as much information as possible about local types to improve the language server

Goal is to keep as much information as possible about local types to
improve the language server
@aboeglin
Copy link
Contributor Author

aboeglin commented Jun 8, 2024

Like we can see in failing tests, it changes the nature of errors. In one case it is less precise which is not good.

I think another option is just to re-run typechecking in case it failed and discard errors but keep as much type info as possible. So typechecking should get like an extra parameter like dry run or similar. So I could keep the original errors and have the type information computed instead/in toSolved. So toSolved would type check again in dry run, discarding errors and catching unification errors.

@aboeglin aboeglin marked this pull request as ready for review June 16, 2024 15:02
@aboeglin aboeglin merged commit 07a0417 into master Jun 16, 2024
2 checks passed
@aboeglin aboeglin deleted the 0.24.0 branch June 16, 2024 15:34
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

Successfully merging this pull request may close these issues.

None yet

1 participant