-
-
Notifications
You must be signed in to change notification settings - Fork 84
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
document updates #2087
base: dev
Are you sure you want to change the base?
document updates #2087
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bump the API and API library patch versions in build/Version.props
/// </summary> | ||
/// <example>Ooopsie, we did a fucky wucky!</example> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oopsie woopsie*
using System.Text.RegularExpressions; | ||
using Microsoft.Extensions.DependencyInjection; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Newline required between system and other usings
@@ -493,7 +493,6 @@ public void Apply(OpenApiDocument swaggerDoc, DocumentFilterContext context) | |||
var productHeaderSchema = new OpenApiSchema | |||
{ | |||
Type = "string", | |||
Format = "productheader", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why was this removed?
Thank you for contributing to tgstation-server! The workflow 'CI Gated PR Comments' requires repository secrets and will not run without approval. Maintainers can add the |
🆑 REST API
Flatten class names.
Add dummy "example" data.
Add some basic description to the authentication methods.
/🆑