Skip to content

Conversation

@teodora-sandu
Copy link
Contributor

@teodora-sandu teodora-sandu commented Mar 26, 2024

Implements the workspace client for this API by using https://github.com/deepmap/oapi-codegen.

The way it works is we have hardcoded the version of the API in the Makefile and use the GitHub PAT provided via an environment variable to download all the specs required from the workspac-service repo. The specs are ignored from git so never get committed. Then, because of oapi-codegen problems when referencing external references (from GitHub) and generating types for external references that do not have a components section, we need to manipulate the specs to follow these conventions.

So make download-apis downloads the specs and manipulates them.
Then make generate generates the types for all the specs and the client for the workspace API.

@github-actions github-actions bot added feature and removed feature labels Mar 26, 2024
@teodora-sandu teodora-sandu force-pushed the feat/workspace-client branch from e5b348f to 1308165 Compare March 26, 2024 17:23
@teodora-sandu teodora-sandu requested a review from cmars March 26, 2024 17:23
@teodora-sandu teodora-sandu marked this pull request as ready for review March 26, 2024 17:23
@teodora-sandu teodora-sandu requested a review from a team as a code owner March 26, 2024 17:23
@teodora-sandu teodora-sandu force-pushed the feat/workspace-client branch from 1308165 to bf1a2c1 Compare March 26, 2024 17:28
@teodora-sandu teodora-sandu changed the title feat: add client for the workspace API feat: add client for the workspace API [IDE-200] Mar 28, 2024
Copy link
Contributor

@bastiandoetsch bastiandoetsch left a comment

Choose a reason for hiding this comment

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

Thanks!

@teodora-sandu teodora-sandu force-pushed the feat/workspace-client branch from 4fcda0c to 3c437ce Compare April 2, 2024 10:58
@teodora-sandu teodora-sandu merged commit 7a465a4 into main Apr 2, 2024
@teodora-sandu teodora-sandu deleted the feat/workspace-client branch April 2, 2024 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants