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

Generate Client using OpenAPI spec tool, Update Sanitations, Update Overview, Setup Guide and Quickstart and Add Test Cases and examples #3

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

MohamedAathif2001
Copy link

@MohamedAathif2001 MohamedAathif2001 commented Jan 8, 2025

Purpose

Examples

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility

@CLAassistant
Copy link

CLAassistant commented Jan 8, 2025

CLA assistant check
All committers have signed the CLA.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
examples/search_taxes/Ballerina.toml Outdated Show resolved Hide resolved
examples/search_taxes/main.bal Outdated Show resolved Hide resolved
Comment on lines 22 to 24

//Search for a tax by label

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
//Search for a tax by label
//Search for a tax by label

Copy link
Author

Choose a reason for hiding this comment

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

Removed the extra line

examples/search_taxes/main.bal Outdated Show resolved Hide resolved
examples/search_taxes/main.bal Outdated Show resolved Hide resolved
@@ -0,0 +1,233 @@
import ballerina/http;
Copy link
Member

Choose a reason for hiding this comment

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

license header?

Copy link
Author

Choose a reason for hiding this comment

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

Added in all the files

MohamedSabthar
MohamedSabthar previously approved these changes Jan 10, 2025
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