❗ Release notes required

✅ Found changes and release notes in following paths:

Change path	Release notes path	Description
src/Compiler	docs/release-notes/.FSharp.Compiler.Service/9.0.300.md

Well - 11 tests, that's much less than I would have expected here :)

Could someone please help me with updating the baselines here? I've tried committing the files produced during the local tests run, but it doesn't seem to succeed on CI.

This does not look like a baseline issue, all the CI runs today had these failures.
See also here.

EDIT: sorry, forget that, seems to be different

Could someone please help me with updating the baselines here? I've tried committing the files produced during the local tests run, but it doesn't seem to succeed on CI.

Hi @auduchinok sorry for the long delay here. I finally got to this - I pushed the changes and it's all green now.

You were actually close. The thing is - there are 2 kinds of baselines here, .bsl and .vsbsl - and this test required adding the latter. Don't ask me why it's like this... artifacts of earlier testing approaches, you know.

@psfinaki Thank you very much!

The tests seem to fail because of #4516. I wish we could finally fix it and not create workarounds in the tooling and tests.

This is ready.

There was some difference between the local run and the CI in how the overload resolution error is reported. This could be because of how FCS tests build project options, which may be somewhat different from the CI. I've removed the diagnostic messages from the new tests for now.