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

#197 Produce AsyncAPI documents at build time (AsyncAPI.Saunter.Generator.Build) #219

Draft
wants to merge 75 commits into
base: main
Choose a base branch
from

Conversation

smoerijf
Copy link
Contributor

@smoerijf smoerijf commented Jul 7, 2024

Ready for review, but depends on PR #217

Nuget package to generate AsyncAPI specs at build time, by injecting .targets file via the nuget package. In the injected post-build target the startup assembly is used to extract the AsyncAPI spec and those files are written to files.

Closes:
#197

Senn Geerts added 30 commits July 6, 2024 14:27
…uments, default all. Added support for env vars
…-global-dotnet-tool' into asyncapi#197-Produce-AsyncAPI-documents-at-build-time

# Conflicts:
#	src/AsyncAPI.Saunter.Generator.Cli/Commands/TofileInternal.cs
…d project to populate the local nuget source
Senn Geerts added 11 commits July 11, 2024 23:43
…-global-dotnet-tool' into asyncapi#197-more-logging

# Conflicts:
#	.github/workflows/ci.yaml
#	.gitignore
#	Saunter.sln
#	src/AsyncAPI.Saunter.Generator.Cli/Commands/TofileInternal.cs
#	src/AsyncAPI.Saunter.Generator.Cli/SwashbuckleImport/CommandRunner.cs
#	src/AsyncAPI.Saunter.Generator.Cli/readme.md
#	test/AsyncAPI.Saunter.Generator.Cli.Tests/AsyncAPI.Saunter.Generator.Cli.Tests.csproj
#	test/AsyncAPI.Saunter.Generator.Cli.Tests/IntegrationTests.cs
#	test/AsyncAPI.Saunter.Generator.Cli.Tests/PackAndInstallLocalTests.cs
#	test/Saunter.Tests/Generation/DocumentGeneratorTests/InterfaceAttributeTests.cs
env:
PATH: ${{ github.env.PATH }}:/home/runner/.dotnet/tools

unit-test:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed separate unittest step because it causes a lot duplicate work:

  • It builds to whole solution again, while the build itself already does this
  • I couldn't get it working to let the test-job depend on the build job build output and use that to run tests.

@smoerijf smoerijf changed the title #197 more logging #197 Produce AsyncAPI documents at build time Jul 13, 2024
@smoerijf smoerijf changed the title #197 Produce AsyncAPI documents at build time #197 Produce AsyncAPI documents at build time (AsyncAPI.Saunter.Generator.Build) Jul 13, 2024
Senn Geerts added 16 commits July 13, 2024 16:47
…at failed to generate specs, fixed now.

Added an external nuget dependancy to the example project (nlog) because that made spec generation fail, fixed now. And now also testing both .NET6 & .NET8
…i#197-more-logging

# Conflicts:
#	Saunter.sln
#	examples/StreetlightsAPI/StreetlightsAPI.csproj
#	src/AsyncAPI.Saunter.Generator.Cli/ToFile/ServiceProviderBuilder.cs
#	src/AsyncAPI.Saunter.Generator.Cli/readme.md
#	test/AsyncAPI.Saunter.Generator.Cli.Tests/AsyncAPI.Saunter.Generator.Cli.Tests.csproj
#	test/AsyncAPI.Saunter.Generator.Cli.Tests/IntegrationTests.cs
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.

1 participant