Release version 4.2

[OsirisTerje]

We have 28 closed issues, and 2 open issues.

The open issues are:

• Switch default branch to main #3981
• Remove some version hard-coding in the OSPlatformTranslator #4577
• Remove reference to the mailing list from CONTRIBUTING.md #4730
• QoL suggestion: fail fast in Assert.Multiple under debugger #4738 PR in progress

It would be awesome if we could manage to close these during the summer, and get a 4.2 out.

[stevenaw]

There's also a few filtering or selection bugs introduced in 4.x which may prove blockers for some to upgrade to NUnit 4. These aren't formally in the milestone at the moment, but we may want to keep these on our radar too

• After upgrade from version 3.14.0 to 4.* running multiple test categories in parentheses separated with 'OR' stopped working #4651
• Exception when using test filters from .runsettings or --filter argument from dotnet test #4589
• Nunit 4.0.x Test case selection is incorrect with certain test data #4584

The first two of these are similar enough that they may have a common root cause

[OsirisTerje]

Agree, we should try to do those too, for 4.2

[maettu-this]

Please note the inconsistency among merged docs PR 932 and pending framework PR 4709 as previoulsy mentioned on Jun 18, asking for support on May 13 and again on Jun 18. In case version 4.2 decides to go without the ProgressTraceListener, the docs changes should be reverted. Also note some considerations.

[OsirisTerje]

@stevenaw @manfred-brands @mikkelbu @SeanKilleen

It seems we are pretty close now. Are there anything else you want to have included ?
Or can I start doing the release process of 4.2 ?

[stevenaw]

Nothing from me, thanks @OsirisTerje

[stevenaw]

I think the ProgressWriter PR is pretty close to merging, just an issue with an unused using I think. Unsure if you wanted to wait on your end or not

[manfred-brands]

Nothing from me either. Thanks @OsirisTerje

[manfred-brands]

I think the ProgressWriter PR is pretty close to merging, just an issue with an unused using I think. Unsure if you wanted to wait on your end or not

I pushed a commit to remove the unnecessary using and @stevenaw merged the PR.