-
Notifications
You must be signed in to change notification settings - Fork 7
feat: add devcontainer templates #1010
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a devcontainer template for C++ to enable easier project setup and distribution. The template packages the existing C++ devcontainer as a reusable template that can be discovered and used by other projects.
Key Changes:
- Adds a new
cpp-templatedevcontainer template with configuration and documentation files - Configures automated publishing of devcontainer templates via GitHub Actions
- Integrates template versioning with the existing release-please workflow
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
release-please-config.json |
Adds extra-files configuration to track template version |
.github/workflows/continuous-integration.yml |
Adds new job to publish devcontainer templates as OCI artifacts |
.devcontainer/cpp-template/devcontainer-template.json |
Defines template metadata and configuration |
.devcontainer/cpp-template/.github/dependabot.yml |
Provides Dependabot configuration for template users |
.devcontainer/cpp-template/.devcontainer/devcontainer.json |
Basic devcontainer configuration referencing Dockerfile |
.devcontainer/cpp-template/.devcontainer/Dockerfile |
Simple Dockerfile that extends the base C++ container image |
✅
|
| Descriptor | Linter | Files | Fixed | Errors | Warnings | Elapsed time |
|---|---|---|---|---|---|---|
| ✅ ACTION | actionlint | 20 | 0 | 0 | 0.52s | |
| ✅ DOCKERFILE | hadolint | 2 | 0 | 0 | 0.78s | |
| ✅ GHERKIN | gherkin-lint | 6 | 0 | 0 | 2.33s | |
| ✅ JSON | npm-package-json-lint | yes | no | no | 0.46s | |
| ✅ JSON | prettier | 19 | 4 | 0 | 0 | 0.57s |
| ✅ JSON | v8r | 19 | 0 | 0 | 8.08s | |
| ✅ MARKDOWN | markdownlint | 11 | 0 | 0 | 0 | 0.9s |
| ✅ MARKDOWN | markdown-table-formatter | 11 | 0 | 0 | 0 | 0.23s |
| ✅ REPOSITORY | gitleaks | yes | no | no | 1.15s | |
| ✅ REPOSITORY | git_diff | yes | no | no | 0.01s | |
| ✅ REPOSITORY | grype | yes | no | no | 30.21s | |
| ✅ REPOSITORY | secretlint | yes | no | no | 1.0s | |
| ✅ REPOSITORY | syft | yes | no | no | 2.11s | |
| ✅ REPOSITORY | trivy | yes | no | no | 4.93s | |
| ✅ REPOSITORY | trivy-sbom | yes | no | no | 0.25s | |
| ✅ REPOSITORY | trufflehog | yes | no | no | 3.13s | |
| lychee | 77 | 1 | 0 | 11.56s | ||
| ✅ YAML | prettier | 28 | 0 | 0 | 0 | 1.4s |
| ✅ YAML | v8r | 28 | 0 | 0 | 7.97s | |
| ✅ YAML | yamllint | 28 | 0 | 0 | 1.29s |
Detailed Issues
⚠️ SPELL / lychee - 1 error
[IGNORED] docker://pandoc/extra:3.7.0@sha256:a703d335fa237f8fc3303329d87e2555dca5187930da38bfa9010fa4e690933a | Unsupported: Error creating request client: builder error for url (docker://pandoc/extra:3.7.0@sha256:a703d335fa237f8fc3303329d87e2555dca5187930da38bfa9010fa4e690933a)
[IGNORED] https://vscode.dev/redirect?url=vscode://ms-vscode-remote.remote-containers/cloneInVolume?url=https://github.com/philips-software/amp-devcontainer | Unsupported: Error creating request client: builder error for url (vscode://ms-vscode-remote.remote-containers/cloneInVolume?url=https://github.com/philips-software/amp-devcontainer)
[500] https://api.securityscorecards.dev/projects/github.com/philips-software/amp-devcontainer/badge | Network error: Internal Server Error
📝 Summary
---------------------
🔍 Total..........126
✅ Successful.....123
⏳ Timeouts.........0
🔀 Redirected.......0
👻 Excluded.........0
❓ Unknown..........0
🚫 Errors...........1
Errors in README.md
[500] https://api.securityscorecards.dev/projects/github.com/philips-software/amp-devcontainer/badge | Network error: Internal Server Error
See detailed reports in MegaLinter artifacts
Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)
- Documentation: Custom Flavors
- Command:
npx [email protected] --custom-flavor-setup --custom-flavor-linters ACTION_ACTIONLINT,DOCKERFILE_HADOLINT,GHERKIN_GHERKIN_LINT,JSON_V8R,JSON_PRETTIER,JSON_NPM_PACKAGE_JSON_LINT,MARKDOWN_MARKDOWNLINT,MARKDOWN_MARKDOWN_TABLE_FORMATTER,REPOSITORY_GIT_DIFF,REPOSITORY_GITLEAKS,REPOSITORY_GRYPE,REPOSITORY_SECRETLINT,REPOSITORY_SYFT,REPOSITORY_TRIVY,REPOSITORY_TRIVY_SBOM,REPOSITORY_TRUFFLEHOG,SPELL_LYCHEE,YAML_PRETTIER,YAML_YAMLLINT,YAML_V8R
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 10 out of 10 changed files in this pull request and generated no new comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
EkelmansPh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool stuff!
|
Pull Request Report (#1010)Static measures
Time related measures
Status check related measures
|
|
🎉 Hooray! The changes in this pull request went live with the release of v6.6.0 🎉 |




🚀 Hey, I have created a Pull Request
Description of changes
This pull request introduces two new devcontainer templates for C++ and Rust, along with supporting configuration files and CI/CD automation for publishing and maintaining these templates. The changes are grouped into template additions, automation enhancements, and configuration improvements.
Devcontainer Template Additions:
Added a C++ devcontainer template:
.devcontainer/cpp-template/devcontainer-template.jsondescribing the template, options, and metadata.devcontainer.jsonfor building the C++ development environment. [1] [2]Added a Rust devcontainer template:
.devcontainer/rust-template/devcontainer-template.jsonwith Rust-specific metadata and options.devcontainer.jsonfor building the Rust development environment. [1] [2]Automation and CI/CD Enhancements:
Configuration and Maintenance Improvements:
release-please-config.jsonto ensure version bumps in the new devcontainer template files are tracked and released automatically..mega-linter.ymlto exclude the new template Dockerfiles from Dockerfile linting.✔️ Checklist