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

add hide option for command options #695

Merged
merged 1 commit into from
Dec 5, 2023
Merged

add hide option for command options #695

merged 1 commit into from
Dec 5, 2023

Conversation

hugomontero
Copy link
Contributor

@hugomontero hugomontero commented Dec 4, 2023 •
edited
Loading

Description

This PR will add the ability to hide some options in help command

How to Test

  1. Pull down the branch: git pull && git checkout feature/sc-123965/hide-options-in-cli
  2. Install dependencies: npm i
  3. Run unit tests: npm run test:unit
  4. run --help option for every logic-function command
  5. Attempt to use all known options for each command.

Outcome

  • Tests should pass
  • options marked with hidden should not appear in help
  • all options will be used

Related Issues / Discussions

Story details: https://app.shortcut.com/particle/story/123965

Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA
  • Problem and solution clearly stated
  • Tests have been provided
  • Docs have been updated
  • CI is passing

@hugomontero hugomontero force-pushed the feature/sc-123396/implement-execute-command-with-eventdata branch from fa201f4 to 1312a53 Compare December 5, 2023 16:15
Base automatically changed from feature/sc-123396/implement-execute-command-with-eventdata to feature/logic-function December 5, 2023 17:12
@hugomontero hugomontero force-pushed the feature/sc-123965/hide-options-in-cli branch from da25047 to a882f1d Compare December 5, 2023 17:15
@hugomontero hugomontero merged commit 661b8f4 into feature/logic-function Dec 5, 2023
5 of 6 checks passed
@hugomontero hugomontero deleted the feature/sc-123965/hide-options-in-cli branch December 5, 2023 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@monkbroc monkbroc monkbroc approved these changes

@keeramis keeramis Awaiting requested review from keeramis

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@hugomontero @monkbroc