equinix / equinix-sdk-go

Golang client for Equinix APIs
https://deploy.equinix.com/labs/equinix-sdk-go
MIT License
2 stars 6 forks source link

feat: API Sync by GitHub Action for @ctreatma #10

Closed github-actions[bot] closed 9 months ago

github-actions[bot] commented 9 months ago

This API Sync PR was triggered by @ctreatma through GitHub Actions workflow_displatch on 2023-12-13.

displague commented 9 months ago

non directly related, but this PR suggests that we'll need a way to generate sync PRs for each and every spec (perhaps all in one job)

ctreatma commented 9 months ago

The service-specific sync job is created automatically when the service is onboarded. The current job structure is based on the opinion that we should expressly not sync all specs in one PR, because one PR that changes multiple services will require massive coordination just to review it. On top of that, it's not entirely clear how broadly applicable a GitHub Actions job for spec syncing is. Metal needs it because the spec URL is unversioned, and it looks like LBaaS will need it for the same reason, at least initially. Fabric is pulled from SwaggerHub using a versioned URL, so it's debatable whether a sync job is actually useful for that.

More generally, it's possible the best outcome is for the sync job to go away completely. In my opinion the ideal process is for the SDK to be updated as part of the API development process rather than after production deployment; then teams can use a branch build of the SDK to validate that their changes work the way customers need them to work.

github-actions[bot] commented 9 months ago

This PR is included in version 0.31.0 :tada: