Skip to content

Latest commit

 

History

History
126 lines (86 loc) · 8.4 KB

CONTRIBUTING.md

File metadata and controls

126 lines (86 loc) · 8.4 KB

Contributing to OpenAPI.net

OpenAPI.net is a mono-repo containing source code for the following packages:

Libraries

Library NuGet Release
Microsoft.OpenAPI NuGet Version
Microsoft.OpenAPI.YamlReader NuGet Version
Microsoft.OpenAPI.Hidi NuGet Version

OpenAPI.net is open to contributions. There are a couple of different recommended paths to get contributions into the released version of this library.

NOTE A signed a contribution license agreement is required for all contributions, and is checked automatically on new pull requests. Please read and sign the agreement before starting any work for this repository.

File issues

The best way to get started with a contribution is to start a dialog with the owners of this repository. Sometimes features will be under development or out of scope for this SDK and it's best to check before starting work on contribution. Discussions on bugs and potential fixes could point you to the write change to make.

Submit pull requests for bug fixes and features

Feel free to submit a pull request with a linked issue.

Branches and support policy

Because one major consumer of these libraries is ASP.net, the support policy of this repository is aligned with dotnet support policy.

The following table outlines the mapping between package major versions, dotnet versions, and which contributions are accepted. As a consumer, make sure the version of this library your application is using is aligned with the version of ASP.net described in the table below.

Major version Branch Supported AspNetCore OpenAPI versions Supported Swashbuckle.AspNetCore version Supported OpenAPI versions Changes provided by Microsoft Accepted contributions End of support date
1.X support/v1 < 10 < 10 2.0, 3.0 security fixes security and bugfixes .NET 9 (Nov 2026)
2.X support/v2 = 10 ¹ = 10 ³ 2.0, 3.0, 3.1 security and bugfixes security and bugfixes .NET 10 (Nov 2028) ¹
3.X ² main not available not available 2.0, 3.0, 3.1, 3.2 security, bugfixes and feature improvements security, bugfixes and feature improvements TBD

Notes:

  1. This assumes that AspNetCore OpenAPI version 11 and above will adopt version 3 or above of this library, otherwise, it'd expand the support date for version 2 of this library.
  2. This will be conditioned by new releases of OpenAPI, this library, ASP.NET and AspNetCore OpenAPI's adoption of new versions of this library.
  3. This assumes that Swashbuckle.AspNetCore version 11 and above will adopt version 3 or above of this library.

Multi-versions requirement for contributions

When contributing to the library, start by making a contribution to the main branch first, or the uppermost version it applies to. During the review process you'll be asked to demonstrate your contribution cannot apply to prior versions or to port your contribution to the branches for prior versions before the initial pull request can get merged.

This approach helps maintain a similar behavior across all versions under active support.

Commit message format

To support our automated release process, pull requests are required to follow the Conventional Commit format. Each commit message consists of a header, an optional body and an optional footer. The header is the first line of the commit and MUST have a type (see below for a list of types) and a description. An optional scope can be added to the header to give extra context.

<type>[optional scope]: <short description>
<BLANK LINE>
<optional body>
<BLANK LINE>
<optional footer(s)>

The recommended commit types used are:

  • feat for feature updates (increments the minor version)
  • fix for bug fixes (increments the patch version)
  • perf for performance related changes e.g. optimizing an algorithm
  • refactor for code refactoring changes
  • test for test suite updates e.g. adding a test or fixing a test
  • style for changes that don't affect the meaning of code. e.g. formatting changes
  • docs for documentation updates e.g. ReadMe update or code documentation updates
  • build for build system changes (gradle updates, external dependency updates)
  • ci for CI configuration file changes e.g. updating a pipeline
  • chore for miscallaneous non-sdk changesin the repo e.g. removing an unused file

Adding an exclamation mark after the commit type (feat!) or footer with the prefix BREAKING CHANGE: will cause an increment of the major version.

Updates to public API surface

Because we need to maintain a compatible public API surface within a major version, this project is using the public API analyzers to ensure no prior public API is changed/removed inadvertently.

This means that:

  • All entries in an Unshipped document need to be moved to the Shipped document before any public release.
  • All new APIs being added need to be Unshipped document before the pull request can be merged, otherwise build will fail with a message like the example below.
Error: /home/runner/work/OpenAPI.NET/OpenAPI.NET/src/Microsoft.OpenApi/Models/OpenApiSecurityScheme.cs(39,46): error RS0016: Symbol 'OAuth2MetadataUrl.set' is not part of the declared public API (https://github.com/dotnet/roslyn-analyzers/blob/main/src/PublicApiAnalyzers/PublicApiAnalyzers.Help.md) [/home/runner/work/OpenAPI.NET/OpenAPI.NET/src/Microsoft.OpenApi/Microsoft.OpenApi.csproj::TargetFramework=net8.0]

Update the unshipped document

To update the unshipped document, simply run the following commands

# add the missing public api entries
dotnet format --diagnostics RS0016
# discard changes to cs files to avoid creating conflicts
git checkout *.cs

Move items from unshipped to unshipped document

. ./scripts/promoteUnshipped.ps1

Note: the promotion of APIs is automated through the dedicated workflow and should result in pull requests being automatically opened.

Updating the benchmark information

To ensure performance of the library does not degrade over time, we have continuous benchmarks running. You might see the continuous integration failing if your pull request changed any model under src/Microsoft.OpenApi/Models.

Benchmark result for EmptyApiSchema does not match the existing benchmark result (original!=new). Allocated bytes differ: 408 != 416

To update the benchmarks, run the following script:

cd performance/benchmark
dotnet run -c Release

Then commit the report files using a "chore" commit.