Skip to content

Conversation

@Yazanmy
Copy link

@Yazanmy Yazanmy commented Nov 17, 2025

  • Read docroot from .ddev/config.yaml
  • Support both docroot and web directories
  • Prevent unbound variable errors in scripts

The Issue

How This PR Solves The Issue

Manual Testing Instructions

ddev add-on get https://github.com/Vardot/ddev-dev-tools/tarball/refs/pull/REPLACE_ME_WITH_THIS_PR_NUMBER/head
ddev restart

Automated Testing Overview

Release/Deployment Notes

- Read docroot from .ddev/config.yaml
- Support both docroot and web directories
- Prevent unbound variable errors in scripts
Copy link
Collaborator

@abdullahajam abdullahajam left a comment

Choose a reason for hiding this comment

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

I truly appreciate your contribution @Yazanmy. But for now, we will keep the files I/O out of the equation, and keep the awk away from spawning extra processes on the server, as we have a very limited resources on it.

We will stick with having docroot or web directories by checking them manually.

@abdullahajam abdullahajam marked this pull request as ready for review November 17, 2025 15:30
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.

Support dynamic project root in file search script

2 participants