Skip to content

Fix devcontainer configuration drift#232

Merged
csweichel merged 1 commit intomainfrom
ak/fix-devcontainer-configuration-drift
Jun 6, 2025
Merged

Fix devcontainer configuration drift#232
csweichel merged 1 commit intomainfrom
ak/fix-devcontainer-configuration-drift

Conversation

@akosyakov
Copy link
Member

Description

  • Remove inaccessible shfmt feature that was causing build failures
  • Specify exact runtime versions (Go 1.24, Node 18) for consistency
  • Move leeway installation from postStartCommand to Dockerfile for proper global dependency management
  • Add port forwarding configuration (8080, 3000, 9000) with descriptions
  • Create comprehensive automations file with development workflows:
    • install-deps: Download Go dependencies (auto-triggered)
    • build-leeway: Build main application (manual)
    • test: Run Go tests (manual)
    • lint: Run linter if available (manual)
  • Remove obsolete install-leeway.sh script

Fixes dev container build failure and establishes proper development environment following Gitpod development setup guidelines.

Related Issue(s)

Fixes #

How to test

Documentation

/hold

- Remove inaccessible shfmt feature that was causing build failures
- Specify exact runtime versions (Go 1.24, Node 18) for consistency
- Move leeway installation from postStartCommand to Dockerfile for proper global dependency management
- Add port forwarding configuration (8080, 3000, 9000) with descriptions
- Create comprehensive automations file with development workflows:
  - install-deps: Download Go dependencies (auto-triggered)
  - build-leeway: Build main application (manual)
  - test: Run Go tests (manual)
  - lint: Run linter if available (manual)
- Remove obsolete install-leeway.sh script

Fixes dev container build failure and establishes proper development environment
following Gitpod development setup guidelines.
"ghcr.io/devcontainers/features/aws-cli:1": {}
},
"postStartCommand": "./.devcontainer/install-leeway.sh"
"forwardPorts": [8080, 3000, 9000],
Copy link
Member Author

Choose a reason for hiding this comment

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

8080 port I can find

"ghcr.io/devcontainers/features/aws-cli:1": {},
"ghcr.io/devcontainers/features/aws-cli:1": {}
},
"postStartCommand": "./.devcontainer/install-leeway.sh"
Copy link
Member Author

Choose a reason for hiding this comment

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

we ask the maintainer install global tools in Dockerfile for caching, I do wonder whether it applies here

"ghcr.io/devcontainers/features/node:1": {},
"ghcr.io/devcontainers/features/go:1": {},
"ghcr.io/devcontainers/features/common-utils:2": {},
"ghcr.io/devcontainers-contrib/features/shfmt:1": {
Copy link
Member Author

Choose a reason for hiding this comment

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

such feature did not exist, was a bug, the agent could not find shfmt usage in the repo, and dropped

@csweichel csweichel merged commit 3e46a85 into main Jun 6, 2025
6 checks passed
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.

3 participants