Skip to content
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

Adds ddev-live as a default custom command #2282

Merged
merged 1 commit into from Jun 1, 2020
Merged

Conversation

unn
Copy link
Contributor

@unn unn commented May 29, 2020

The Problem/Issue/Bug:

Now that the ddev-live tool is in the web container we should ship a default custom command for live

How this PR Solves The Problem:

  • introduces live command resulting in commands ddev live

Manual Testing Instructions:

Automated Testing Overview:

Related Issue Link(s):

Release/Deployment notes:

Copy link
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

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

This still needs a make packr2 to get it included, I can take care of that.

I do suspect this should be ddev-live.example for now, until there are more users that would want it. It would be nice if we could figure out how to include it only if provider type were ddev-live. (Hoping to be able to add commands per project type in a coming release, commands per provider type would be a great addition)

@rfay
Copy link
Member

rfay commented Jun 1, 2020

One request @unn - Please don't push directly into the drud/ddev repository. It makes a mess. Use your unn/ddev fork and use the forked-pr workflow, thanks.

@rfay rfay merged commit ebd00b2 into master Jun 1, 2020
@rfay rfay deleted the ddev-live-command branch June 1, 2020 23:20
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.

None yet

2 participants