Skip to content

Conversation

@mxr576
Copy link
Contributor

@mxr576 mxr576 commented Sep 22, 2025

The Issue

#42

How This PR Solves The Issue

The optimized version reduces 8 individual shell spawns to 3-5 executions by intelligently batching related operations. The most significant improvement combines the repository setup and DDEV installation into a single shell execution, eliminating the major source of process overhead.

Critical operations like version holding and autostart remain as separate executions to maintain proper error handling and directory context, ensuring reliability isn't compromised for performance gains.

Manual Testing Instructions

Automated Testing Overview

Related Issue Link(s)

Release/Deployment Notes

@mxr576
Copy link
Contributor Author

mxr576 commented Sep 22, 2025

Results are still mixed on the first run 😢 But it was promising on the first 3 runs.

https://github.com/mxr576/github-action-setup-ddev/actions/workflows/main.yml

image image

@mxr576
Copy link
Contributor Author

mxr576 commented Sep 22, 2025

https://github.com/mxr576/github-action-setup-ddev/actions/runs/17910391441/usage -> DDEV install maximum: 1m 37s.
https://github.com/mxr576/github-action-setup-ddev/actions/runs/17910424651/job/50920513675 -> DDEv install maximum: 1m 33s.

So the current changes seems to have an impact... but could we reduce all billing times to 1 minutes, consistenty? 🤔

@mxr576
Copy link
Contributor Author

mxr576 commented Sep 22, 2025

@mxr576
Copy link
Contributor Author

mxr576 commented Sep 22, 2025

Closing in favor of #44 that actually solves the increased billing problem.

@mxr576 mxr576 closed this Sep 22, 2025
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.

1 participant