Skip to content

docs: reduce custom code#1173

Draft
Ron (rjaegers) wants to merge 11 commits intomainfrom
docs/reduce-custom-code
Draft

docs: reduce custom code#1173
Ron (rjaegers) wants to merge 11 commits intomainfrom
docs/reduce-custom-code

Conversation

@rjaegers
Copy link
Member

@rjaegers Ron (rjaegers) commented Feb 28, 2026

🚀 Hey, I have created a Pull Request

Description of changes

✔️ Checklist

  • I have followed the contribution guidelines for this repository
  • I have added tests for new behavior, and have not broken any existing tests
  • I have added or updated relevant documentation
  • I have verified that all added components are accounted for in the SBOM

Copilot AI review requested due to automatic review settings February 28, 2026 13:35
@rjaegers Ron (rjaegers) requested a review from a team as a code owner February 28, 2026 13:35
@rjaegers Ron (rjaegers) marked this pull request as draft February 28, 2026 13:35
@github-actions
Copy link
Contributor

github-actions bot commented Feb 28, 2026

📦 Container Size Analysis

Note

Comparing ghcr.io/philips-software/amp-devcontainer-base:edgeghcr.io/philips-software/amp-devcontainer-base:pr-1173

📈 Size Comparison Table

OS/Platform Previous Current Change Trend
linux/amd64 175.16 MB 175.16 MB 263 B (0%) 🔽
linux/arm64 167.63 MB 167.63 MB 212 B (0%) 🔽

@github-actions
Copy link
Contributor

github-actions bot commented Feb 28, 2026

⚠️MegaLinter analysis: Success with warnings

Descriptor Linter Files Fixed Errors Warnings Elapsed time
✅ ACTION actionlint 23 0 0 0.62s
✅ DOCKERFILE hadolint 3 0 0 0.83s
✅ JSON npm-package-json-lint yes no no 0.49s
✅ JSON prettier 21 4 0 0 0.51s
✅ JSON v8r 21 0 0 7.69s
✅ MARKDOWN markdownlint 12 0 0 0 1.2s
✅ MARKDOWN markdown-table-formatter 12 0 0 0 0.26s
✅ REPOSITORY checkov yes no no 17.64s
✅ REPOSITORY gitleaks yes no no 0.62s
✅ REPOSITORY git_diff yes no no 0.01s
✅ REPOSITORY grype yes no no 30.68s
✅ REPOSITORY secretlint yes no no 0.92s
✅ REPOSITORY syft yes no no 2.01s
✅ REPOSITORY trivy yes no no 5.49s
✅ REPOSITORY trivy-sbom yes no no 0.25s
✅ REPOSITORY trufflehog yes no no 2.29s
⚠️ SPELL lychee 83 1 0 21.91s
✅ YAML prettier 31 0 0 0 0.99s
✅ YAML v8r 31 0 0 8.84s
✅ YAML yamllint 31 0 0 1.02s

Detailed Issues

⚠️ SPELL / lychee - 1 error
[IGNORED] docker://pandoc/extra:3.9.0.0-ubuntu@sha256:72afa9c8d3300e5f10c9c4330e101725687f2179bffd912fb859c6d2ae85de62 | Unsupported: Error creating request client: builder error for url (docker://pandoc/extra:3.9.0.0-ubuntu@sha256:72afa9c8d3300e5f10c9c4330e101725687f2179bffd912fb859c6d2ae85de62)
[403] https://developer.arm.com/downloads/-/arm-gnu-toolchain-downloads | Network error: Forbidden
[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)
📝 Summary
---------------------
🔍 Total..........126
✅ Successful.....123
⏳ Timeouts.........0
🔀 Redirected.......0
👻 Excluded.........0
❓ Unknown..........0
🚫 Errors...........1

Errors in .github/TOOL_VERSION_ISSUE_TEMPLATE.md
[403] https://developer.arm.com/downloads/-/arm-gnu-toolchain-downloads | Network error: Forbidden

See detailed reports in MegaLinter artifacts

You could have the same capabilities but better runtime performances if you use a MegaLinter flavor:

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 mega-linter-runner@9.3.0 --custom-flavor-setup --custom-flavor-linters ACTION_ACTIONLINT,DOCKERFILE_HADOLINT,JSON_V8R,JSON_PRETTIER,JSON_NPM_PACKAGE_JSON_LINT,MARKDOWN_MARKDOWNLINT,MARKDOWN_MARKDOWN_TABLE_FORMATTER,REPOSITORY_CHECKOV,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

MegaLinter is graciously provided by OX Security

Copy link
Contributor

Copilot AI left a 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 updates the documentation generation pipeline to rely more directly on SBDL tooling (instead of custom Python conversion), embedding SBDL model content alongside existing Gherkin feature files.

Changes:

  • Embed SBDL aspect/requirement declarations into compatibility.feature using @sbdl-begin / @sbdl-end blocks.
  • Update the SRS Jinja template to render aspects and their referenced requirements from the SBDL model.
  • Remove the custom Gherkin→SBDL Python conversion scripts and update the reusable document-generation workflow to compile via sbdl -m compile.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
test/cpp/features/compatibility.feature Adds embedded SBDL model blocks for compatibility requirements.
docs/templates/software-requirements-specification.md.j2 Adjusts template logic to render aspects + requirements from the compiled SBDL model.
docs/support/gherkin_sbdl_converter.py Removes custom converter implementation.
docs/support/gherkin_mapping_config.py Removes custom mapping configuration.
docs/support/gherkin-to-sbdl.py Removes CLI wrapper for the custom converter.
.github/workflows/wc-document-generation.yml Switches to sbdl -m compile and updates dependencies/artifacts for doc generation.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 28, 2026

📦 Container Size Analysis

Note

Comparing ghcr.io/philips-software/amp-devcontainer-rust:edgeghcr.io/philips-software/amp-devcontainer-rust:pr-1173

📈 Size Comparison Table

OS/Platform Previous Current Change Trend
linux/amd64 555.58 MB 555.57 MB 261 B (0%) 🔽
linux/arm64 509.75 MB 509.75 MB 336 B (0%) 🔽

@github-actions
Copy link
Contributor

github-actions bot commented Feb 28, 2026

📦 Container Size Analysis

Note

Comparing ghcr.io/philips-software/amp-devcontainer-cpp:edgeghcr.io/philips-software/amp-devcontainer-cpp:pr-1173

📈 Size Comparison Table

OS/Platform Previous Current Change Trend
linux/amd64 696.86 MB 696.86 MB 489 B (0%) 🔽
linux/arm64 677.67 MB 677.67 MB 322 B (0%) 🔽

@github-actions
Copy link
Contributor

github-actions bot commented Feb 28, 2026

Test Results

 13 files  ±0   13 suites  ±0   20m 45s ⏱️ +45s
 33 tests ±0   33 ✅ ±0  0 💤 ±0  0 ❌ ±0 
137 runs  ±0  137 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit e06a4b7. ± Comparison against base commit 02927aa.

♻️ This comment has been updated with latest results.

@rjaegers Ron (rjaegers) temporarily deployed to acceptance-testing March 2, 2026 14:07 — with GitHub Actions Inactive
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

@rjaegers Ron (rjaegers) temporarily deployed to acceptance-testing March 3, 2026 09:38 — with GitHub Actions Inactive
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.

2 participants