dotnet / dotnet/fsharp

Remove dictionary order dependence in error ranges.

Open
#10,342 0 comments 0 reactions 0 assignees View on GitHub
Area-Testing Impact-Low
Dominant language
F#
Stars
4.3k
Forks
876
Avg merge
4d 11h
Merged PRs (30d)
131

Description

This PR: https://github.com/dotnet/fsharp/pull/10308

changed the implementation from using Dictionary to using ConcurrentDictionary.

A side effect is that some tests error range changed, we should ensure that we are not reliant on dictionary storage order.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.