Conversation
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
❌MegaLinter analysis: Error
Detailed Issues❌ GHERKIN / gherkin-lint - 1 error
|
There was a problem hiding this comment.
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.featureusing@sbdl-begin/@sbdl-endblocks. - 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. |
| description is | ||
| "amp-devcontainer *SHALL* be compatible with both the x86-64 (AMD64) *and* AArch64 (ARM64) host architectures." | ||
| remark is | ||
| "Supporting both x86-64 and AArch64 has several advantages:\n\n- Increasing useability on a wide range of host machines, from PC hardware using the x86-64 architecture to Apple Silicon using the AArch64 architecture\n- Unlocking the power efficiency of the AArch64 architecture, potentially reducing energy consumption and cost for metered ci-systems" |
There was a problem hiding this comment.
Typo in the requirement remark text: "useability" should be "usability".
| "Supporting both x86-64 and AArch64 has several advantages:\n\n- Increasing useability on a wide range of host machines, from PC hardware using the x86-64 architecture to Apple Silicon using the AArch64 architecture\n- Unlocking the power efficiency of the AArch64 architecture, potentially reducing energy consumption and cost for metered ci-systems" | |
| "Supporting both x86-64 and AArch64 has several advantages:\n\n- Increasing usability on a wide range of host machines, from PC hardware using the x86-64 architecture to Apple Silicon using the AArch64 architecture\n- Unlocking the power efficiency of the AArch64 architecture, potentially reducing energy consumption and cost for metered ci-systems" |
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
📦 Container Size AnalysisNote Comparing 📈 Size Comparison Table
|
The @sbdl markers were seen as tags
|




🚀 Hey, I have created a Pull Request
Description of changes
✔️ Checklist