JohnCampionJr / Finbuckle.MultiTenant.MongoFramework

MongoFramework store and per tenant data isolation for Finbucle.MultiTenant
MIT License
4 stars 1 forks source link

Upgrade/include .NET 6 #1

Closed rangers-globecar closed 2 years ago

rangers-globecar commented 2 years ago

@JohnCampionJr Hi John, Please accept this PR to include .NET 6

Please note that I did this, as in a recent project migration, I tried to use your library, and was giving binding error, pointing to MongoFramework 0.25.0.

In this PR:

  1. includes "multi-target" adjustements, added .NET6, and replace NETCORE2.1 by NETSTARDARD2.0
  2. multi-target testings, NetCoreApp2.1, 3,1, NET5, NET6.
  3. Updated Samples to NET 6 (that is why so many files)

N.B. support for NetCoreApp2.1 is out of scope, please consider dropping support for it... Other libraries like MongoFramwork 0.26.x have stopped supporting it. There minimum compatibility is NETSTANDARD2.1

rangers-globecar commented 2 years ago

Hi @JohnCampionJr, Do you have any comments or thinks you would like to edit ? Let me know.

JohnCampionJr commented 2 years ago

Hi, thanks for this. We need to update the GH workflow to test with .NET 6.0. I'll take a look at it ASAP unless you have time.

rangers-globecar commented 2 years ago

I sumitted a correction.... Can you release the pipeline ?

Or give me the permission to run them?