Skip to content

Fix Swagger errors#1498

Merged
neolynx merged 2 commits intoaptly-dev:masterfrom
chesseed:fix/swagger-errors
Oct 19, 2025
Merged

Fix Swagger errors#1498
neolynx merged 2 commits intoaptly-dev:masterfrom
chesseed:fix/swagger-errors

Conversation

@chesseed
Copy link
Copy Markdown
Contributor

@chesseed chesseed commented Oct 9, 2025

Fixes #

Requirements

All new code should be covered with tests, documentation should be updated. CI should pass.

Description of the Change

This fixes three small errors in the generated swagger file

  1. Path parameters are never optional
  2. Remove duplicate return annotation
  3. Remove duplicated path parameter

Checklist

  • unit-test added (if change is algorithm)
  • functional test added/updated (if change is functional)
  • man page updated (if applicable)
  • bash completion updated (if applicable)
  • documentation updated
  • author name in AUTHORS

@neolynx neolynx self-assigned this Oct 19, 2025
@neolynx neolynx self-requested a review October 19, 2025 18:29
Copy link
Copy Markdown
Member

@neolynx neolynx left a comment

Choose a reason for hiding this comment

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

thanks !

@neolynx neolynx merged commit 4526d6d into aptly-dev:master Oct 19, 2025
37 checks passed
@chesseed chesseed deleted the fix/swagger-errors branch December 5, 2025 15:58
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