Skip to content

Conversation

@NoahGorny
Copy link
Member

Cleanup composer completion as part of #1782

@cornfeedhobo
Copy link
Member

cornfeedhobo commented Jan 28, 2021

It'd be great to get someone to manually test this ... also to tell me what composer is.

@NoahGorny
Copy link
Member Author

It'd be great to get someone to manually test this ... also to tell me what composure is.

composer is some kind of dependency manager for php, see link.
I did test this manually, but the alias stuff is indeed weird.. I will take another look
thank you @cornfeedhobo !!

@NoahGorny NoahGorny force-pushed the cleanup/composer-completions branch from 4fb522d to 0ebc067 Compare February 5, 2021 22:19
@NoahGorny NoahGorny force-pushed the cleanup/composer-completions branch from 0ebc067 to 0d963b2 Compare February 5, 2021 22:23
local cur script coms opts com
COMPREPLY=()
_get_comp_words_by_ref -n : cur words
function _composer() {
Copy link
Member

@cornfeedhobo cornfeedhobo Feb 5, 2021

Choose a reason for hiding this comment

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

@NoahGorny Could you change this to __composer_complete or __composer_completion?

Copy link
Member Author

Choose a reason for hiding this comment

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

done

@NoahGorny NoahGorny merged commit 25c1c89 into Bash-it:master Feb 6, 2021
@NoahGorny NoahGorny deleted the cleanup/composer-completions branch February 6, 2021 21:36
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.

4 participants