-
-
Notifications
You must be signed in to change notification settings - Fork 585
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
project type filter for ddev list
command (ddev list --type=
)
#4613
Conversation
Download the artifacts for this pull request: |
What do you think @cmuench , does this have a future? |
@rfay The filter is already working. I had a bigger idea but we can start with this approach. That's why it's currently marked as draft. |
0b833df
to
ad85d26
Compare
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.
This is looking quite good and is a major code improvement, thanks!
It's missing
- the proper updates to docs to mention the new feature, https://ddev.readthedocs.io/en/latest/users/usage/commands/#list
- It needs to have examples add in the command so
ddev help list
will show this option (and perhaps others).
ddev list
command (ddev list --type=
)
Done |
Proper naming Co-authored-by: Matt Stein <m@ttste.in>
it's done |
Yay, thanks. Back now and will catch up on PRs eventually :) |
No pressure. I was not sure if I had to inform you. |
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.
Thanks!
The Issue
As a user of many different local installed ddev projects I want to filter the
ddev list
output. Currently we have theonly-active
flag available.Additionally we could filter by project type.
How This PR Solves The Issue
The PR restructures the parameters of the list command into a struct. This prepares the command to get additional filters in the future like tags.
Manual Testing Instructions
Run the
ddev list
command with new filter option-t
or--type
and set a project typeExample:
ddev list -t php
orddev list -t magento2
.The filter can be combined with active only filter.
ddev list -A -t php
Automated Testing Overview
I added a simple test which filters by a not existing type. Currently I am looking for a good way to test also the project type filter by an existing type.