ThreeMammals / Ocelot

.NET API Gateway
https://www.nuget.org/packages/Ocelot
MIT License
8.32k stars 1.63k forks source link

Ocelot to .NET 8 #1787

Closed ggnaegi closed 10 months ago

ggnaegi commented 10 months ago

Expected Behavior / New Feature

.NET 8 was launched this week, so we expect Ocelot to be compatible with .NET 8 as soon as possible. ๐Ÿ˜ธ

Actual Behavior / Motivation for New Feature

Only supporting .NET 7

Specifications

raman-m commented 10 months ago

โš ๏ธโš ๏ธโš ๏ธ โ—Importantโ— Feature branch is release/net8. It is created from main. Delivery process is based on Gitflow Hotfix one. That means, this feature (upgrade) should be delivered first, before Oct'23!

raman-m commented 10 months ago

Tests and examples are targeting .NET8

I'm not sure about this. If the lib targets multiple frameworks then why should tests target only one framework?

ggnaegi commented 10 months ago

Ok, let's begin from scratch again, to summarize:

raman-m commented 10 months ago

During last month I have been thinking about release strategy. On some day we will have to decide on versioning and release strategy. It seems today is this day...

The old school release strategy was

But when .NET 6 provided multi targeted frameworks feature everything was changed, and release strategy could be more simplest if changes are .NET versioning only. Also, git repo back porting doesn't make sense anymore. Support can be organized using one repo and one release branch, having one code base. But in this modern approach we have to use more preprocessor directives.

Let's try this approach and we will see how it's useful... This release strategy requires less resources.

raman-m commented 10 months ago

Implemented and delivered by #1790