Add AOT compatibility for SDK

jamesmcroft commented 4 months ago

Fixes #25

This pull request includes significant changes related to the CI/CD workflows, project structure, and codebase of the SwaggerMerge SDK. The most significant changes are the addition of new CI workflows, removal of an old CI workflow, the addition of AOT compatibility tests, and various modifications to the SDK's codebase.

Continuous Integration and Deployment:

.github/workflows/ci-aot.yml: A new CI workflow has been added to check for AOT (Ahead-of-Time) compatibility. This workflow runs on the main branch and on any pull requests targeting main. It checks the src, tests, build directories, and the workflow file itself.
.github/workflows/ci-sdk.yml: Another new CI workflow has been added, which seems to be a more general CI workflow for the SDK. It also runs on the main branch and on any pull requests targeting main. It checks the same directories as the AOT workflow.

AOT Compatibility:

build/Test-AotCompat.ps1: A new PowerShell script has been added to test AOT compatibility. The script accepts a target .NET framework version as a parameter, runs the AOT compatibility tests, and checks for any warnings.

Codebase Modifications:

src/SwaggerMerge.SDK/Common/Extensions/CollectionExtensions.cs: A new extension class has been added to the SDK that provides additional methods for collections.
src/SwaggerMerge.SDK/Configuration/Input/SwaggerInputConfiguration.cs: The File property in the SwaggerInputConfiguration class has been made nullable.
src/SwaggerMerge.SDK/Document/SwaggerDocument.cs: Several changes have been made to the SwaggerDocument class, including making some properties nullable, modifying the OnDeserialized and OnSerializing methods, and adding new constructors to the SwaggerDocumentDefinitions class.

Project Structure:

SwaggerMerge.sln: The solution file has been modified to include a new test project for AOT compatibility. The project configurations have also been updated accordingly.
Directory.Build.props: The StyleCop.Analyzers package reference has been removed from the project file.

PR checklist

[x] Code styling has been met on new source file changes
[ ] Contains NO breaking changes

Other information

guillaume-chervet commented 2 months ago

hi @jamesmcroft do you have any news ? :)

jamesmcroft commented 1 month ago

Hi @guillaume-chervet, apologies for the long delay!

I've been working on this again today to resolve the AOT compatibility issues which I believe are now resolved by migrating from Newtonsoft to System JSON libraries.

I'll validate this and publish a preview of this package soon.

jamesmcroft / swagger-merge