Skip to content

Commit f7de410

Browse files
Copilottrask
andcommitted
Address PR feedback: Add volume mount for Gradle cache, rename README, and remove redundant GRADLE_OPTS
Co-authored-by: trask <[email protected]>
1 parent a541c37 commit f7de410

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ This devcontainer configuration is designed to optimize the development environm
55
## What's Included
66

77
- **Java 17 (Temurin)**: Pre-installed Java Development Kit
8-
- **Gradle 8.11.1**: Pre-installed build tool matching project requirements
8+
- **Gradle Wrapper**: Uses project's gradle wrapper for build consistency
99
- **Docker in Docker**: For containerized smoke tests
1010
- **VS Code Extensions**: Java development extensions
1111

@@ -24,4 +24,4 @@ This devcontainer is automatically used by GitHub Copilot agents when working on
2424

2525
## Validation
2626

27-
Run `.github/scripts/validate-devcontainer.sh` to verify the setup is working correctly.
27+
Run `.devcontainer/validate-devcontainer.sh` to verify the setup is working correctly.

.devcontainer/devcontainer.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,11 @@
2020
}
2121
},
2222
"containerEnv": {
23-
"GRADLE_OPTS": "-Dorg.gradle.daemon=true -Dorg.gradle.parallel=true -Dorg.gradle.caching=true -XX:MaxMetaspaceSize=512m"
23+
"GRADLE_OPTS": "-Dorg.gradle.daemon=true"
2424
},
25+
"mounts": [
26+
"source=gradle-cache,target=/home/vscode/.gradle,type=volume"
27+
],
2528
"postCreateCommand": "bash .devcontainer/setup-devcontainer.sh",
2629
"remoteUser": "vscode",
2730
"// comment": "This devcontainer pre-installs Java 17 and Docker-in-Docker. The remoteUser sets the default user for the container. The setup script builds the project to pre-download dependencies and populate the build cache, reducing initial build time for Copilot agents from 5+ minutes to under 1 minute"

0 commit comments

Comments
 (0)