Skip to content

Conversation

@gitressa
Copy link
Contributor

This is awesome, thanks! It's great that something with so many moving parts and manual configuration can be easily installed and just works.

I do think that emphasizing that a restart is required would be great. I installed it and got an error when I immediately after that tried running a test, and just got an error.

Also, a header would be nice.

The Issue

How This PR Solves The Issue

Manual Testing Instructions

Automated Testing Overview

Related Issue Link(s)

Release/Deployment Notes

Add header and emphasize that a restart is required.
@weitzman weitzman merged commit 659fb2e into ddev:main Dec 23, 2023
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.

2 participants