-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Launch with ddev adminer
instead of launcha -a
#16
Conversation
Thank you for the PR @mejta . I believe the original idea with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this and think it's a good idea, but it doesn't seem like it accomplishes anything to remove launcha
. If you leave launcha
in there at least for a year or so, it will help people to adapt.
This can be tested manually with |
@mejta in the future, you'll be a lot happier creating a named branch for PRs. That way, the next time you create a PR (or need the main branch for something else) life will be a lot better. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, tested manually and it works right. It's a lot better than the launcha
approach. Let's remember to remove launcha in a few months?
I opened But... as I was writing it I thought maybe we'd be better to just go ahead and change |
Thank you, @rfay, for approving the request. Next time, I will create a separate branch for that. Also, I will add an event to remove launcha in February :) |
The Issue
The current way to launch Adminer is via
ddev launcha -a
. It's different from how e.g. PHPMyAdmin is launched (ddev phpmyadmin
).How This PR Solves The Issue
PR changes launching the Adminer the same way like PHPMyAdmin:
ddev adminer
Manual Testing Instructions
Run
ddev adminer