Skip to content

when no worker is defined, do not stop the complete stack - #398

Merged
lyrixx merged 1 commit into
mainfrom
fix/stop-workers-when-no-worker
May 29, 2026
Merged

when no worker is defined, do not stop the complete stack#398
lyrixx merged 1 commit into
mainfrom
fix/stop-workers-when-no-worker

Conversation

@xavierlacot

Copy link
Copy Markdown
Member

When no worker is defined in the "worker" profile, running castor docker:worker:stop stops the complete stack. This PR instead displays an error message in such a case.

@xavierlacot xavierlacot added the enhancement New feature or request label May 20, 2026

@pyrech pyrech left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just two minor comments

Comment thread .castor/docker.php Outdated
Comment thread .castor/docker.php Outdated

@lyrixx lyrixx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Like it 👍🏼 Thanks.

I wait for the pyrech's comments to be fixed, then merge it

@lyrixx
lyrixx force-pushed the fix/stop-workers-when-no-worker branch 2 times, most recently from f337efd to 7646590 Compare May 29, 2026 12:20

@lyrixx lyrixx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@xavierlacot I finished the PR.

PHPStan is ko, but not related. I'll pass another PR before this one

@lyrixx
lyrixx force-pushed the fix/stop-workers-when-no-worker branch from 7646590 to 955e8b2 Compare May 29, 2026 12:48
@lyrixx
lyrixx merged commit 5d93733 into main May 29, 2026
4 checks passed
@lyrixx
lyrixx deleted the fix/stop-workers-when-no-worker branch May 29, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants