Remove support for .NET Standard 2.0 and 2.1

[ErikEJ]

fixes #2340

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.05%. Comparing base (c33bde2) to head (470141b).

❗ Current head 470141b differs from pull request most recent head a5c75f5. Consider uploading reports for the commit a5c75f5 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2386      +/-   ##
==========================================
- Coverage   72.69%   71.05%   -1.64%     
==========================================
  Files         313      296      -17     
  Lines       61737    60725    -1012     
==========================================
- Hits        44878    43150    -1728     
- Misses      16859    17575     +716     

Flag	Coverage Δ
addons	92.88% <ø> (ø)
netcore	73.65% <ø> (-3.38%)	⬇️
netfx	70.12% <ø> (-0.24%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ErikEJ]

@JRahnama All net462 builds pass, despite being labelled with netstandard2.1 - but maybe the label is misleading?

[JRahnama]

@JRahnama All net462 builds pass, despite being labelled with netstandard2.1 - but maybe the label is misleading?

labels are misleading, I will try to make a test pipeline for this PR today.

Update: failures are all due to building netstandard tests which are expected.

[ErikEJ]

@JRahnama any progress on the pipeline Update?

[ErikEJ]

@David-Engel @JRahnama Is this still on the radar?

[DavoudEshtehari]

A new file with .NET Standard support has been added in pull request #2310.

[ErikEJ]

I will merge with latest main and remove it.

[ErikEJ]

How is it going with the pipeline update??

[DavoudEshtehari]

We're going to set up a test pipeline to verify the outcome of this change. Once it's merged, we'll replace it with the current CI.

[ErikEJ]

@DavoudEshtehari sounds great. I will Update tomorrow

[DavoudEshtehari]

You should be able to see this pipeline, which has been setup for your branch.

[DavoudEshtehari]

LGTM! In addition to the CI pipeline, we recently pushed a few YAML files for production pipeline into the GitHub repo, and they should be updated and verified internally. We'll prepare the necessary internal changes to ensure that these PRs are merged with the least time gap.

[ErikEJ]

@DavoudEshtehari Thanks, let me know if there is anything else I need to do