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

feat(clap_complete): add native completion support for elvish #5544

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

shannmu
Copy link
Contributor

@shannmu shannmu commented Jun 21, 2024

Related issue #3919

Work in this PR

  • Add elvish glue script code to support that we could call external native completion.
  • Add the same level of tests as fish

@epage
Copy link
Member

epage commented Jun 21, 2024

related issue: #3919

If this isn't closing out the issue, what is left?

@shannmu
Copy link
Contributor Author

shannmu commented Jun 21, 2024

If this isn't closing out the issue, what is left?

I don't understand what you mean. Adding the related issue is for easier tracking :)

@epage
Copy link
Member

epage commented Jun 21, 2024

I don't understand what you mean. Adding the related issue is for easier tracking :)

The language used doesn't make this sound like the issue is fully resolved by this PR, so I was asking what is left. In fact, there are standardized terms for fully resolving an issue as Github has support for keywords to link PRs to issues (not just backlinks) and even auto-close issues.

@epage
Copy link
Member

epage commented Jul 10, 2024

FYI this now has merge conflicts

@epage
Copy link
Member

epage commented Jul 17, 2024

New set of conflicts to resolve

@epage epage merged commit ff20480 into clap-rs:master Jul 18, 2024
22 checks passed
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.

None yet

2 participants