Skip to content

TA-4038: Implement beta-tagging commands and options#241

Merged
Zgjim Haziri (ZgjimHaziri) merged 4 commits into
masterfrom
TA-4038-beta-tag-commands
Aug 4, 2025
Merged

TA-4038: Implement beta-tagging commands and options#241
Zgjim Haziri (ZgjimHaziri) merged 4 commits into
masterfrom
TA-4038-beta-tag-commands

Conversation

@ZgjimHaziri

@ZgjimHaziri Zgjim Haziri (ZgjimHaziri) commented Jul 31, 2025

Copy link
Copy Markdown
Contributor

Description

  • Added beta tagging by:
    • .beta() method when defining the command. This sets the whole command as beta.
    • .betaOption() for defining an option that will be in beta. This defines only the option as beta.
  • The above methods, add a custom isBeta property to the Command/Option object.
  • Implemented custom Help extension which checks the beta flag of the option or the command when printing them in the help dialog. If it's beta, it prints a (beta) warn postfix.
  • Added printing of a warning info message when the command/options are used outside of help.
    • In options, a message will be printed for every beta option used.
    • Option warnings will not be shown if the command itself is in beta, because of redundancy. Only the command warning will be printed.

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

Relevant links

Jira issue: https://celonis.atlassian.net/browse/TA-4038

Screenshots

Beta command invocation message:
Screenshot 2025-08-01 at 15 35 45

Beta option invocation message:
Screenshot 2025-08-01 at 15 36 54

Beta command in --help:
Screenshot 2025-08-01 at 15 37 51

Beta option in --help:
Screenshot 2025-08-01 at 15 38 21

@ZgjimHaziri Zgjim Haziri (ZgjimHaziri) requested a review from a team as a code owner July 31, 2025 21:03
Buqeta
Buqeta (Buqeta) previously approved these changes Aug 1, 2025

@promeris Meris Nici (promeris) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please include some screenshot in description of the changes in action. Looks good!

@Buqeta

Copy link
Copy Markdown
Contributor

Please include some screenshot in description of the changes in action. Looks good!

https://youtu.be/yKNxeF4KMsY?t=81 😄

Comment thread src/core/command/module-handler.ts Outdated
Comment thread src/core/command/module-handler.ts
Comment thread src/core/command/module-handler.ts Outdated
@ZgjimHaziri

Copy link
Copy Markdown
Contributor Author

Please include some screenshot in description of the changes in action. Looks good!

Added the screenshots

@ZgjimHaziri Zgjim Haziri (ZgjimHaziri) merged commit 7cd5ee6 into master Aug 4, 2025
1 check passed
@ZgjimHaziri Zgjim Haziri (ZgjimHaziri) deleted the TA-4038-beta-tag-commands branch August 4, 2025 08:26
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.

5 participants