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

Sync with dotnet/fsharp 2021-11-12 #8

Conversation

alfonsogarciacaro
Copy link

@alfonsogarciacaro alfonsogarciacaro commented Nov 14, 2021

@ncave I think you prefer rebasing, but it was difficult to fix all the conflicts so the PR includes some merge commits. Also, I removed ParseFile from service_slim because I didn't need and I wasn't sure how to fix the build errors, but please feel free to re-add it for completenees (in that case, probably ParseAndCheckFileInProject should take sourceReader: string -> int * Lazy<string> as parameter instead of the full list of sources).

Besides that and the changes to keep up with FCS I just parallelized the parsing phase in ParseAndCheckProject.

@ncave
Copy link
Owner

ncave commented Nov 16, 2021

@alfonsogarciacaro Closing this, it was merged separately. Sorry I couldn't keep your commit as is because of the rebasing, but I attributed you in the commit message. Thanks for your help!

@ncave ncave closed this Nov 16, 2021
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.

3 participants