devshop icon indicating copy to clipboard operation
devshop copied to clipboard

Dothooks file no longer supported?

Open chadamas opened this issue 1 year ago • 2 comments

@jonpugh Is the Dothooks approach no longer the preferred way of executing deployment commands on the 1.x branch? The module is still there, but there doesn't appear to be a way to enable it in the project or environment settings. I did patch a workaround into our Devshop install, and it seems to function pretty much the same as it did in earlier versions, but if it's going to be removed entirely at some point, I'd rather not do that.

I haven't had much luck using the deploy steps in the composer.json file--the deploy:build step throws an error on new site installations, almost like it's trying to fire before the site is actually installed (interestingly, if the site is already built, the deploy:build step appears to work). I've also tried adding some custom scripts to Composer's standard events (e.g. post-install-cmd), but I'm not sure how best to get the site alias to use for Drush commands going that route.

If you have any advice or info you'd be willing to share, I'd really appreciate it! Thank you.

chadamas avatar Jan 22 '24 17:01 chadamas

Heya, I really appreciate hearing from you. I don't even know how much devshop is used. I've been ready to give it up.

It's been very painful to develop and support, so I've been trimming things down, perhaps too much.

Your right. I shouldn't remove dothooks. I should just mark it deprecated.

A .hooks.yml file is still a possibility, but we should go about it in a different way.

If you can submit a PR to revert the removal of devshop_dothooks, I would merge it. Otherwise, I will get to it at some point.

Thanks!

jonpugh avatar Jan 26 '24 15:01 jonpugh

This issue has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

stale[bot] avatar Jul 19 '25 05:07 stale[bot]