ci: improvements to speed and robustness #954
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR improves CI pipeline efficiency and robustness by optimizing build triggers, enhancing test reporting, and updating dependencies.
- Removes unnecessary builds on main branch pushes now that merge queue is enabled
- Integrates acceptance test results into the unified test reporting system
- Updates Node.js to latest version in acceptance test workflow
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/continuous-integration.yml | Removes push trigger on main branch to prevent unnecessary builds |
| .github/workflows/wc-build-push.yml | Optimizes Docker tag generation to only build edge tags during merge queue events |
| .github/workflows/wc-acceptance-test.yml | Updates Node.js version and configures JUnit test result output |
| .github/workflows/wc-integration-test.yml | Updates artifact naming for consistency with acceptance tests |
| test/cpp/features/playwright.config.ts | Configures JUnit reporter for CI environments |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Contributor
✅
|
| Descriptor | Linter | Files | Fixed | Errors | Warnings | Elapsed time |
|---|---|---|---|---|---|---|
| ✅ ACTION | actionlint | 20 | 0 | 0 | 0.61s | |
| ✅ DOCKERFILE | hadolint | 2 | 0 | 0 | 0.78s | |
| ✅ GHERKIN | gherkin-lint | 2 | 0 | 0 | 1.13s | |
| ✅ JSON | npm-package-json-lint | yes | no | no | 0.52s | |
| ✅ JSON | prettier | 15 | 2 | 0 | 0 | 0.54s |
| ✅ JSON | v8r | 15 | 0 | 0 | 8.54s | |
| ✅ MARKDOWN | markdownlint | 11 | 0 | 0 | 0 | 0.94s |
| ✅ MARKDOWN | markdown-table-formatter | 11 | 0 | 0 | 0 | 0.24s |
| ✅ REPOSITORY | gitleaks | yes | no | no | 1.05s | |
| ✅ REPOSITORY | git_diff | yes | no | no | 0.01s | |
| grype | yes | no | 1 | 28.37s | ||
| ✅ REPOSITORY | secretlint | yes | no | no | 1.05s | |
| ✅ REPOSITORY | syft | yes | no | no | 2.02s | |
| ✅ REPOSITORY | trivy | yes | no | no | 5.09s | |
| ✅ REPOSITORY | trivy-sbom | yes | no | no | 0.25s | |
| ✅ REPOSITORY | trufflehog | yes | no | no | 2.54s | |
| ✅ SPELL | lychee | 65 | 0 | 0 | 3.23s | |
| ✅ YAML | prettier | 26 | 0 | 0 | 0 | 0.9s |
| ✅ YAML | v8r | 26 | 0 | 0 | 8.55s | |
| ✅ YAML | yamllint | 26 | 0 | 0 | 1.08s |
Detailed Issues
⚠️ REPOSITORY / grype - 1 warning
warning: A medium vulnerability in python package: urllib3, version 1.26.20 was found at: /.devcontainer/cpp/requirements.txt
warning: 1 warnings emitted
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
Contributor
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
Contributor
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
Contributor
…ips-software/amp-devcontainer into ci/reduce-build-duplication
Co-authored-by: Copilot <[email protected]> Signed-off-by: Ron <[email protected]>
|
andjordan
approved these changes
Sep 30, 2025
Contributor
Pull Request Report (#954)Static measures
Time related measures
Status check related measures
|
Contributor
|
🎉 Hooray! The changes in this pull request went live with the release of v6.5.0 🎉 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚀 Hey, I have created a Pull Request
Description of changes
This PR improves on some aspects of ci:
✔️ Checklist