Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Json improvements #48

Merged
merged 10 commits into from
Dec 1, 2024
Merged

Json improvements #48

merged 10 commits into from
Dec 1, 2024

Conversation

dlmelendez
Copy link
Contributor

  • Remove Newtonsoft, add System.Text.Json with System.Net.Http.Json
  • Add MetaData based json code generation for better performance with .net6 and higher
  • Added tests for the json code generation, tests for all used requests and responses in the client and in the library namespaces
  • Fixed tests to not catch assert errors
  • Corrected .netstandard to use .net8 libraries, will allow users to use lts .net8 or .net9.
  • Resolved test project security issues by adding Microsoft.Extensions.Hosting latest 8.x version
  • Updated LangVersion to 13.0 - most current
  • Deleted HttpClientExtensions.cs
  • Increased version to 3.1

@ahwm
Copy link
Owner

ahwm commented Dec 1, 2024

This looks great so far! I'll check a few other things and run some additional tests and I'll let you know.

Thanks!

@ahwm ahwm merged commit b957bd4 into ahwm:master Dec 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants