Skip to content

Hide hidden subcommands from completions#443

Merged
rauhul merged 1 commit intomainfrom
hide-hidden-subcommands-from-completions
Apr 24, 2022
Merged

Hide hidden subcommands from completions#443
rauhul merged 1 commit intomainfrom
hide-hidden-subcommands-from-completions

Conversation

@rauhul
Copy link
Copy Markdown
Collaborator

@rauhul rauhul commented Apr 23, 2022

  • Adds a filter to prevent hidden subcommands from appearing in shell
    completion scripts.

Fixes #442, rdar://92182881

- Adds a filter to prevent hidden subcommands from appearing in shell
  completion scripts.

Fixes #442, rdar://92182881
@rauhul rauhul requested a review from natecook1000 April 23, 2022 04:10
@rauhul
Copy link
Copy Markdown
Collaborator Author

rauhul commented Apr 23, 2022

@swift-ci please test

Copy link
Copy Markdown
Member

@natecook1000 natecook1000 left a comment

Choose a reason for hiding this comment

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

LGTM! 👏

@rauhul rauhul merged commit 4f63cee into main Apr 24, 2022
@rauhul rauhul deleted the hide-hidden-subcommands-from-completions branch April 24, 2022 01:56
leuski pushed a commit to leuski/swift-argument-parser that referenced this pull request Jun 17, 2022
* 'main' of github.com:apple/swift-argument-parser: (114 commits)
  Fix `AsyncParseableCommand` hierarchy (apple#436)
  Add experimental manual page generation (apple#332)
  Improving edit distance string extension (apple#446)
  List valid options in error messages for enum array argument (apple#445)
  Remove LinuxMain.swift (apple#367)
  Hide hidden subcommands from completions (apple#443)
  Update changelog for 1.1.2 release (apple#441)
  Fix error message for @option array without values (apple#435)
  Fix Repeat's endless printing (apple#437)
  build: statically link ArgumentParserToolInfo always (apple#424)
  Update changelog for the 1.1.1 release (apple#428)
  build: complete the changes from apple#423 (apple#425)
  Remove platform requirement from Package.swift (apple#427)
  build: repair the build after apple#404 (apple#423)
  Fix broken links/incorrect variance calculation (apple#422)
  Update changelog for the 1.1.0 release (apple#421)
  Update documentation (apple#420)
  Make `@OptionGroup(visibility:)` a public API (apple#419)
  Support an `async` entry point for commands (apple#404)
  Fix a typo and template links (apple#418)
  ...
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.

Completion scripts include completion for hidden subcommands, effectively unhiding them

2 participants