Skip to content

feat(init): Setup chrome service on docker-compose - #408

Merged
lyrixx merged 1 commit into
jolicode:mainfrom
loic425:add-chrome-on-init
Jun 1, 2026
Merged

feat(init): Setup chrome service on docker-compose#408
lyrixx merged 1 commit into
jolicode:mainfrom
loic425:add-chrome-on-init

Conversation

@loic425

@loic425 loic425 commented May 28, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread .castor/init.php
* Insert the Chrome service before the "volumes:" section if it exists.
* Otherwise, append it to the end of the file.
*/
if (preg_match('/^volumes:/m', $content, $matches, PREG_OFFSET_CAPTURE)) {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It has been done by ChatGPT, so it needs to be checked...

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.

tell us when it's ready

@loic425 loic425 May 29, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ok, I've challenged chatGPT to explain this line and the regex option and this is legit 👍

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.

I meant, have you really tested it? I underdstand the code, but it does not mean it's work well :)

@loic425 loic425 Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I meant, have you really tested it? I underdstand the code, but it does not mean it's work well :)

Yes I've tried it and run it twice to check and everything is ok 👍

@loic425
loic425 force-pushed the add-chrome-on-init branch from e02a0ae to 3fb42c7 Compare May 28, 2026 10:02
Comment thread README.md
@loic425
loic425 force-pushed the add-chrome-on-init branch from 3fb42c7 to c36e187 Compare May 28, 2026 10:07
@lyrixx
lyrixx merged commit cc87f05 into jolicode:main Jun 1, 2026
4 checks passed
@lyrixx

lyrixx commented Jun 1, 2026

Copy link
Copy Markdown
Member

thanks

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.

2 participants