cobra icon indicating copy to clipboard operation
cobra copied to clipboard

feat(completion): support no-space completions for short flags

Open mazitovt opened this issue 2 years ago • 10 comments

Close #1629

mazitovt avatar Jun 17 '22 07:06 mazitovt

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jun 17 '22 07:06 CLAassistant

@marckhouzam pls review the PR

mazitovt avatar Jun 17 '22 17:06 mazitovt

Duplicate of #1727?

Luap99 avatar Jun 18 '22 09:06 Luap99

Thanks for the contribution @mazitovt !

Duplicate of #1727?

As @Luap99 noticed we just got a similar contribution in #1727. I've started reviewing that one and we should be getting it merged soon. I'll still have a look at yours as it may give a different perspective.

marckhouzam avatar Jun 18 '22 14:06 marckhouzam

@marckhouzam Thanks!

mazitovt avatar Jun 18 '22 14:06 mazitovt

@mazitovt I've had a look at this PR and it is a step in the right direction. It is very similar to #1727 which came in first, so I'll focus on #1727 for now.

One note for your info is that although the Go logic works with your PR, the actual shell completion does not work as expected because the different shell script need to be updated to support this usecase. Note that #1727 has the same problem, but we'll work at getting it resolved.

marckhouzam avatar Jun 21 '22 15:06 marckhouzam

The Cobra project currently lacks enough contributors to adequately respond to all PRs. This bot triages issues and PRs according to the following rules:

  • After 60d of inactivity, lifecycle/stale is applied. - After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied and the PR is closed. You can:
  • Make a comment to remove the stale label and show your support. The 60 days reset. - If a PR has lifecycle/rotten and is closed, comment and ask maintainers if they'd be interseted in reopening.

github-actions[bot] avatar Sep 07 '22 00:09 github-actions[bot]

Reopening. I plan on reviewing this. The other PR was closed by the author.

marckhouzam avatar Oct 07 '22 01:10 marckhouzam

The Cobra project currently lacks enough contributors to adequately respond to all PRs. This bot triages issues and PRs according to the following rules:

  • After 60d of inactivity, lifecycle/stale is applied. - After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied and the PR is closed. You can:
  • Make a comment to remove the stale label and show your support. The 60 days reset. - If a PR has lifecycle/rotten and is closed, comment and ask maintainers if they'd be interested in reopening.

github-actions[bot] avatar Dec 08 '22 00:12 github-actions[bot]

The CLA is not signed @mazitovt

marckhouzam avatar Oct 13 '23 19:10 marckhouzam