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.

Sign up for GitHub

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

Jump to bottom

[Improvement]: Remove the tool option validation warning when there are no tool options #42704

Closed
TharmiganK opened this issue May 8, 2024 · 0 comments · Fixed by #42748
Closed

[Improvement]: Remove the tool option validation warning when there are no tool options #42704

TharmiganK opened this issue May 8, 2024 · 0 comments · Fixed by #42748
Assignees
@gayaldassanayake
Labels
Area/CLI Ballerina CLI related issues. Team/DevTools Ballerina Developer Tooling ( CLI, Test FW, Package Management, OpenAPI, APIDocs ) Type/Improvement

Comments

@TharmiganK
Copy link
Contributor

Description

$Subject

Describe your problem(s)

With the Ballerina tool integration, we can define tool configurations in the Ballerina.toml file. The tool options for the OpenAPI tool are optional. If the developer has not provided any tool options, we are printing the following warning:

WARNING: Validation of tool options of 'openapi' for 'oas_client_sentiment_oas' is skipped due to no tool options found

But this warning does not make sense since there is no tool options hence there is no actual requirement to validate any of them

Describe your solution(s)

Remove this warning

Related area

-> Other Area

Related issue(s) (optional)

No response

Suggested label(s) (optional)

No response

Suggested assignee(s) (optional)

No response
@ballerina-bot ballerina-bot added the needTriage The issue has to be inspected and labeled manually label May 8, 2024
@gayaldassanayake gayaldassanayake added Area/CLI Ballerina CLI related issues. Team/DevTools Ballerina Developer Tooling ( CLI, Test FW, Package Management, OpenAPI, APIDocs ) and removed needTriage The issue has to be inspected and labeled manually labels May 8, 2024
@gayaldassanayake gayaldassanayake self-assigned this May 14, 2024
@gayaldassanayake gayaldassanayake mentioned this issue May 14, 2024
Merged
13 tasks
@gayaldassanayake gayaldassanayake mentioned this issue May 15, 2024
Merged
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gayaldassanayake gayaldassanayake

Labels
Area/CLI Ballerina CLI related issues. Team/DevTools Ballerina Developer Tooling ( CLI, Test FW, Package Management, OpenAPI, APIDocs ) Type/Improvement
Projects
Ballerina Team Main Board
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[master] Remove warning for no required options in tool schema gayaldassanayake/ballerina-lang
3 participants
@ballerina-bot @gayaldassanayake @TharmiganK