fix(devcontainer): remove unknown flag on docker-compose

[EdamAme-x]

Closes #3581

The author should do the following, if applicable

• Add tests
• Run tests
• bun run format:fix && bun run lint:fix to format the code
• Add TSDoc/JSDoc to document the code

[EdamAme-x]

What do you think about this?
@marcomuser

I think that there is little security risk with this change.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.71%. Comparing base (e7732a5) to head (0afc681).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3582   +/-   ##
=======================================
  Coverage   94.71%   94.71%           
=======================================
  Files         158      158           
  Lines        9535     9535           
  Branches     2807     2807           
=======================================
  Hits         9031     9031           
  Misses        504      504           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[marcomuser]

Oh sorry about that. I had not tested the new config with workspaces. The additional config values are coming straight from the devcontainer docs: https://code.visualstudio.com/remote/advancedcontainers/docker-options#_podman

[EdamAme-x]

Thanks for your reply.
This change made it work.

[yusukebe]

I'm not familiar with the dev container, but it looks good. @EdamAme-x Is this okay to merge?

[EdamAme-x]

I think it will be fine!

[yusukebe]

LGTM!

[yusukebe]

@EdamAme-x Thanks! Let's go.

[marcomuser]

I tried out your branch locally with Podman and I can still build the container image and run all tests. Seems to work with Podman as well!

[EdamAme-x]

Thanks for testing it.