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

[Bug]: Issues in Ballerina help text #41980

Open
anupama-pathirage opened this issue Jan 16, 2024 · 4 comments · Fixed by ballerina-platform/edi-tools#27
Open

[Bug]: Issues in Ballerina help text #41980

anupama-pathirage opened this issue Jan 16, 2024 · 4 comments · Fixed by ballerina-platform/edi-tools#27

Comments

@anupama-pathirage
Copy link
Contributor

Description

  1. bal tool command is not in help text. - @azinneera related issue Incorrect help text in CLI page ballerina-dev-website#8535
  2. bal health command is not in help text. - @nirmal070125 is this something we missed?
  3. bal edi command has incorrect text. - @chathurace / @niveathika check this.
image

Steps to Reproduce

No response

Affected Version(s)

No response

OS, DB, other environment details and versions

No response

Related area

-> Compilation

Related issue(s) (optional)

No response

Suggested label(s) (optional)

No response

Suggested assignee(s) (optional)

No response

@ballerina-bot ballerina-bot added needTriage The issue has to be inspected and labeled manually userCategory/Compilation labels Jan 16, 2024
@gayaldassanayake
Copy link
Contributor

  1. The tool command is listed under other commands. The website has it under core commands, which should be fixed.
  2. As long as the health tool is installed, the help text for the health command should be listed. Would you be able to run bal tool list and check if health is already installed locally (if not, have to pull it using bal tool pull health). Then it will be visible with bal --help under Tool Commands section.
  3. The edi help text needs to be corrected in https://github.com/ballerina-platform/edi-tools/blob/60ef628b6e59efea1c4efb12f338c3bf8735b98c/edi-cli/src/main/java/io/ballerina/edi/cmd/EdiCmd.java#L37 @niveathika

@gayaldassanayake gayaldassanayake self-assigned this Jan 16, 2024
@gayaldassanayake gayaldassanayake removed the needTriage The issue has to be inspected and labeled manually label Jan 16, 2024
@niveathika
Copy link
Contributor

@RDPerera Could you fix the edi tool text?

@RDPerera
Copy link
Member

RDPerera commented Jan 17, 2024

@RDPerera Could you fix the edi tool text?

Fix the 3rd issue + add bal help edi text with 0.8.6 release. In the meantime, I am working on improving the help text of the EDI tool to match standard Ballerina CLI help texts as well.

Screenshot 2024-01-17 at 10 48 33

Copy link

github-actions bot commented Feb 1, 2024

This issue is NOT closed with a proper Reason/ label. Make sure to add proper reason label before closing. Please add or leave a comment with the proper reason label now.

      - Reason/EngineeringMistake - The issue occurred due to a mistake made in the past.
      - Reason/Regression - The issue has introduced a regression.
      - Reason/MultipleComponentInteraction - Issue occured due to interactions in multiple components.
      - Reason/Complex - Issue occurred due to complex scenario.
      - Reason/Invalid - Issue is invalid.
      - Reason/Other - None of the above cases.

@RDPerera RDPerera reopened this Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants