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

fix: help for extension commands for plugins with hyphens in the name. #1048

Merged
merged 2 commits into from Sep 23, 2021

Conversation

yacchi
Copy link
Contributor

@yacchi yacchi commented Sep 18, 2021

Summary

When a plugin with a hyphen in its name contains an extension command (lib/commands/commands-*), the - in the plugin name displayed by asdf help will be blank.

Fixes: #1034

@yacchi yacchi requested a review from a team as a code owner September 18, 2021 15:42
@yacchi yacchi force-pushed the fix-help-for-extension-command branch 2 times, most recently from 9804f12 to a7164b2 Compare September 18, 2021 15:48
@yacchi yacchi force-pushed the fix-help-for-extension-command branch from a7164b2 to 798f918 Compare September 18, 2021 15:48
@yacchi yacchi changed the title Fix help for extension commands for plugins with hyphens in the name. fix: help for extension commands for plugins with hyphens in the name. Sep 18, 2021
@Stratus3D
Copy link
Member

Thanks for the PR @yacchi ! And thanks for writing a test! It looks like the linter is failing for these changes so we'll have to get that fixed before we merge.

@Stratus3D
Copy link
Member

Not sure why it was saying checks failed before and now it's saying all have passed. I'm going to go ahead and merge this. Thanks for contributing this fix!

@Stratus3D Stratus3D merged commit 3e0cb9a into asdf-vm:master Sep 23, 2021
@yacchi yacchi deleted the fix-help-for-extension-command branch September 23, 2021 19:14
@yacchi
Copy link
Contributor Author

yacchi commented Sep 23, 2021

@Stratus3D
Thanks for merge this pull request!
And, I'm sorry for the confusion about Lint.

At the time I created the pull request, the git commit message and pull request name did not comply with Lint's rules.
The Lint error may be related to this.

While I was looking for a way to rerun CI tasks, it seemed that all CI tasks on Github had gone green.

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.

bug: incorrect help for plugin extensions of hyphenated named plugin
2 participants