-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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
[8.x] Add DB command to drop into the database CLI #35304
Merged
Merged
Conversation
This file contains 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
paras-malhotra
changed the title
[8.x] Add DB command to open the database CLI
[8.x] Add DB command to drop into the database CLI
Nov 20, 2020
Think you can omit that flag and just make it an optional argument: |
@driesvints makes sense, made the change! Also renamed the title of the argument to |
driesvints
reviewed
Nov 20, 2020
driesvints
reviewed
Nov 20, 2020
@driesvints done! 👍 |
driesvints
approved these changes
Nov 20, 2020
dinhquochan
reviewed
Nov 29, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Motivation
I currently use many aliases to connect to my database on my Laravel apps. It would be so much easier if we could just run
php artisan db
orphp artisan db --database=mysql
to quickly drop into the database CLI! No more aliases to remember for every app and easily switch between connections for apps that have multiple connections. This is similar to what Rails has in therails db
command.Screenshots
I've tested this on mysql, pgsql and sqlite. I don't have SQL Server but I think it should run without any errors.