Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Consume linode-api-docs/openapi.json rather than openapi.yaml #727

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

lgarber-akamai
Copy link
Contributor

📝 Description

This pull request re-points the default OpenAPI specification URL from

https://raw.githubusercontent.com/linode/linode-api-docs/.../openapi.yaml

to

https://raw.githubusercontent.com/linode/linode-api-docs/.../openapi.json

in response to the corresponding change in the linode-api-docs repository.

✔️ How to Test

The following test steps assume you have pulled down this PR locally and run make install.

Manual Testing

  1. Run an arbitrary command using the new Linode CLI install.

Integration Testing

make test-int

@lgarber-akamai lgarber-akamai added the repo-ci-improvement for improvements in the repository or CI workflow in the changelog. label Feb 7, 2025
@lgarber-akamai lgarber-akamai requested a review from a team as a code owner February 7, 2025 14:25
@lgarber-akamai lgarber-akamai requested review from zliang-akamai and yec-akamai and removed request for a team February 7, 2025 14:25
Copy link
Contributor

@yec-akamai yec-akamai left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@ezilber-akamai ezilber-akamai left a comment

Choose a reason for hiding this comment

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

Tests passing locally

@lgarber-akamai lgarber-akamai merged commit 184b017 into linode:dev Feb 7, 2025
14 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
repo-ci-improvement for improvements in the repository or CI workflow in the changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants