Skip to content

Refresh the Swift Package Registry OpenAPI document #8096

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

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

czechboy0
Copy link
Member

Motivation:

The OpenAPI document for Swift Package Registry was incomplete and didn't contain all the requirements described in the related Swift Evolution proposals.

Modifications:

Rewrote the OpenAPI document based on the current reading of all the Swift Package Registry Swift Evolution proposals and documentation in this repository.

You can view the rendered version of the docs at: https://editor.swagger.io/?url=https://raw.githubusercontent.com/czechboy0/swift-package-manager/refs/heads/hd-refresh-registry-openapi/Documentation/PackageRegistry/registry.openapi.yaml

Result:

A much more accurate OpenAPI doc, one that can be used for code generation.

@dschaefer2
Copy link
Member

@swift-ci please test

Copy link
Member

@dschaefer2 dschaefer2 left a comment

Choose a reason for hiding this comment

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

Awesome. Thanks @czechboy0!

Co-authored-by: Paul LeMarquand <plemarquand@gmail.com>
@plemarquand
Copy link
Contributor

@swift-ci please test

@plemarquand plemarquand enabled auto-merge (squash) November 4, 2024 19:42
@dschaefer2
Copy link
Member

dschaefer2 commented Nov 4, 2024

So weird why this doesn't happen by default all the time...
@swift-ci please test windows

@dschaefer2
Copy link
Member

@swift-ci please test windows

@plemarquand plemarquand merged commit 1e155f7 into swiftlang:main Nov 4, 2024
5 checks passed
@czechboy0 czechboy0 deleted the hd-refresh-registry-openapi branch November 5, 2024 09:27
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.

4 participants