chore: fix dev container toolchain and give it a stable name

Giannis Kepasrequested to merge
gitlab-community/gitlab-org/api/client-go:dev-container-improvements into main

What does this MR do and why?

  • Fix the dev container so make reviewable works out of the box
    • This was caused by postStartCommand mishandling how eval run in that step
  • Drop the Go feature since mise already provisions Go exactly as .gitlab-ci.yml does, making .tool-versions the single source of truth
  • Enforce a container name. name only labels the VS Code UI, so docker ps showed a generated name like zen_elgamal
  • Bump docker-in-docker to v4
  • Feature versions pinned exactly (docker-in-docker:4.0.0, mise:1.0.0) instead of floating majors, matching .tool-versions
    • gitignore .devcontainer/devcontainer-lock.json since the inline pins make it redundant

Merge request reports