Skip to content
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

Add OpenAPI Specification, Sanitations and Generated Client #3

Merged
merged 4 commits into from
Aug 8, 2024

Conversation

SanduniU
Copy link
Contributor

@SanduniU SanduniU commented Aug 7, 2024

Purpose

Adding OpenAI OpenAPI specification, Adding sanitations and generating the client, types, and utilities for the API

Approach

  • Added the OpenAI OpenAPI spec to the project.
  • Did necessary sanitations on the OpenAPI Specification and documented the sanitations in sanitations.md
  • Generated client code based on the new API specification.
  • Generated type definitions for the updated API endpoints.
  • Generated utility functions to support the new client and type definitions.
  • Changed .gitignore
  • Added Dependencies.toml

@CLAassistant
Copy link

CLAassistant commented Aug 7, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

Choose a reason for hiding this comment

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

@SanduniU IMO we can remove these test skeletons from this PR and, send the completed test suite in a separate PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Test directory was deleted from this branch

Copy link
Contributor

@NipunaRanasinghe NipunaRanasinghe left a comment

Choose a reason for hiding this comment

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

LGTM

@NipunaRanasinghe NipunaRanasinghe merged commit 83344be into ballerina-platform:main Aug 8, 2024
1 of 2 checks passed
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.

3 participants