Update Laravel alias to use the correct path for Composer global binaries.#2275
Closed
adbarbosa wants to merge 1 commit intoBash-it:masterfrom
Closed
Update Laravel alias to use the correct path for Composer global binaries.#2275adbarbosa wants to merge 1 commit intoBash-it:masterfrom
adbarbosa wants to merge 1 commit intoBash-it:masterfrom
Conversation
seefood
approved these changes
Nov 25, 2024
Contributor
seefood
left a comment
seefood
left a comment
There was a problem hiding this comment.
See remark and fix if relevant, or mark it resolved if you are 100% sure it's not. Thanks!
| # A list of useful laravel aliases | ||
|
|
||
| alias laravel='${HOME?}/.composer/vendor/bin/laravel' | ||
| alias laravel='${HOME?}/.config/composer/vendor/bin/laravel' |
Contributor
There was a problem hiding this comment.
Any chance some users still use the old location on older installations and will somehow lose functionality? I have a feeling this can benefit from an "if" and a fallback location if an older installation is found.
seefood
approved these changes
Apr 30, 2025
Contributor
seefood
left a comment
seefood
left a comment
There was a problem hiding this comment.
not perfect, but I'll take it for now.
seefood
pushed a commit
to seefood/bash-it
that referenced
this pull request
May 5, 2025
Contributor
|
implemented more elegantly in #2306 |
seefood
added a commit
to seefood/bash-it
that referenced
this pull request
Aug 26, 2025
* master: (767 commits) Tofu completion rewrite have pre-commit ignore the /vendor, they should be immutable. Fix: __powerline_last_status_prompt to handle unset argument safely remove superfluous function and pick better var names for readability Update base.theme.bash Update base.theme.bash Update base.theme.bash Update base.theme.bash Fix for 2323 issue Apply fixes Add `.git-blame-ignore-revs` Clean themes A-L Update powerline-multiline.base.bash Fix a couple bugs introduced by last commit Fix bad merge Update plugins/available/extract.plugin.bash important syntax correction from the owenr of ble.sh only the correct FZF integration loads, sepending on whether the blesh plugin is enabled as well. readonly HIST* variables is a bit extreme and clashes with ble.sh and other tools. implement Bash-it#2275 more elegantly ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This pull request updates the
laravelalias in Bash-it to reference the correct path for globally installed Composer binaries (~/.config/composer/vendor/bin/laravel). This change ensures that users can execute thelaravel newcommand without needing manual adjustments to their environment'sPATH.Motivation and Context
The default path for Composer global installations has changed from
~/.composerto~/.config/composer. Without this update, users with the default Composer configuration encounter issues when trying to use thelaravelalias, as it points to the outdated path.This change resolves the issue and aligns Bash-it's Laravel alias with Composer's current global installation directory structure.
No open issues were directly linked to this problem at the time of submission.
How Has This Been Tested?
Screenshots (if appropriate):
N/A
Types of changes
Checklist:
clean_files.txtand formatted it usinglint_clean_files.sh.