-
-
Notifications
You must be signed in to change notification settings - Fork 150
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
Update ddev documentation #1356
base: main
Are you sure you want to change the base?
Update ddev documentation #1356
Conversation
Thanks for the update. The “Install DDEV” section could also be simplified. Corresponding instructions are formulated in detail in the ddev documentation. Therefore possibly sufficient: DDEV is available for various platforms. As a »Windows« user, in addition to the WSL2 workflow (recommended), you can also simply use the current Windows installation binaries. |
Add additional information regarding “Create project”. We should mention here that the directory name corresponds to the later “hostname” and that this could also be further configured. |
We should perhaps add a new paragraph regarding “Custom PHP Configuration”. For example, for local changes, e.g. “memory_limit = -1” per project etc.. |
@fkaminski I think the description for PHP settings and hostname is too much documentation as this is a good working example. |
With regard to “as much as necessary, as little as possible”, this would certainly be correct. But: |
@avhulst After local testing, I think it would not be a bad idea to add this note at the end of the “Additional information” section: {{% notice note %}} {{% notice note %}} |
Co-authored-by: Franko <[email protected]>
Co-authored-by: Franko <[email protected]>
Co-authored-by: Franko <[email protected]>
@fkaminski I have added the paragraph 'Custom PHP Configuration' and the 'Git Bash' note for Windows. In my opinion, WSL is sufficiently covered in the DDEV documentation and is too OS specific for me. |
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.
My final comments. @fritzmg / @netzarbeiter Otherwise I think it's great.
Co-authored-by: Franko <[email protected]>
Co-authored-by: Franko <[email protected]>
Co-authored-by: Franko <[email protected]>
Co-authored-by: Franko <[email protected]>
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.
Some typo/formatting (with help of deepl) ...
Co-authored-by: Franko <[email protected]>
Co-authored-by: Franko <[email protected]>
Co-authored-by: Franko <[email protected]>
Co-authored-by: Franko <[email protected]>
Co-authored-by: Franko <[email protected]>
Update documentation for Contao 5.3 and newest DDEV version