FinancialModelingPrep.NET icon indicating copy to clipboard operation
FinancialModelingPrep.NET copied to clipboard

Use Refit to generate API calls

Open Matthiee opened this issue 2 years ago • 7 comments

  • [x] Check if it makes sense to use https://github.com/reactiveui/refit
  • [ ] Refactor to use Refit bindings

Matthiee avatar Aug 10 '22 16:08 Matthiee

I will try to implement this. I have reviewed Refit and it looks VERY impressive.

tforsberg avatar Oct 01 '22 14:10 tforsberg

@tforsberg Sounds good!

Matthiee avatar Oct 01 '22 14:10 Matthiee

I created a branch called "Refit" in my fork. Can I remove the .NET 5 support in this project?

tforsberg avatar Oct 02 '22 19:10 tforsberg

If possible I would like to keep both for now. Is there something that doesn't work in .NET 5 regarding refit?

Matthiee avatar Oct 03 '22 06:10 Matthiee

No. It should all work in .NET 5 also.

The reason I'd like to use .NET 6 only is there are several improvements in the C# version 10 language. https://learn.microsoft.com/en-us/dotnet/csharp/whats-new/csharp-10

.NET 5 uses C# version 9 .NET 6 uses C# version 10

tforsberg avatar Oct 03 '22 12:10 tforsberg

You can remove the .NET 5 support.

Matthiee avatar Oct 03 '22 16:10 Matthiee

Ok.

tforsberg avatar Oct 03 '22 17:10 tforsberg