Skip to content

Improve Polarion vs EA display when model is already in sync#107

Merged
danielsiegl merged 2 commits into
mainfrom
copilot/improve-polarion-display
Apr 7, 2026
Merged

Improve Polarion vs EA display when model is already in sync#107
danielsiegl merged 2 commits into
mainfrom
copilot/improve-polarion-display

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

The ReviewPolarionRequirements.yml workflow posted an unhelpful PR comment with an empty Changed Requirements: section when Polarion was already in sync with the EA model.

Changes

  • ReviewPolarionRequirements.yml — "Post Finish message in PR" step now branches on whether modified requirements exist:
    • No changes: Renders a structured status card (branch, commit, run link) consistent with the UpdatePolarionByComments workflow style
    • Changes present: Preserves existing format with LemonTree session links and requirements list

New "in sync" comment format:

## Polarion vs Model Comparison - Run #<run_id>

**Status:** ✅ Polarion is in sync with EA model

**Details:**
- Branch: `<branch>`
- Changed Requirements: None
- Commit: `<commit-id>`
- Run: [View logs](<url>)

---
*This comment is automatically updated per run*

Copilot AI changed the title [WIP] Improve display for Polarion matches with EA Improve Polarion vs EA display when model is already in sync Apr 7, 2026
Copilot AI requested a review from danielsiegl April 7, 2026 10:16
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 7, 2026

Model Check Results

LemonTree ModelCheck results

Severity Issue Message
🔴 Error Model has 1 suspected traceability links Some requirements have suspected traceability links. Please review and confirm the traceability links.
🔴 Error Model has 1 deleted ALM elements Some ALM elements have been deleted. Please review and confirm the deletions.
🟠 Warning Model has 57 unsatisfied requirements Each requirement should be linked to at least one design element using a 'satisfy' relationship.
🟠 Warning Model has 2 ports without interface definition All ports should have an interface definition assigned.
🟠 Warning Model has 30 untraced Polarion requirements Each Polarion requirement must have at least one Trace relationship (type Trace or stereotype 'trace') to a non-requirement (architecture) element.
🟢 Passed All requirements only have incoming links to architecture elements
🟢 Passed There are no orphaned InterfaceBlocks
🟢 Passed No Journal entries in the model

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 7, 2026

✔️ Model is identical!

Modified Elements in SystemArchitecture:

Found 0 Diagrams.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 7, 2026

Polarion vs Model Comparison - Run #24076227963

Status: ✔️ Polarion is in sync with EA model

Details:

  • Branch: copilot/improve-polarion-display
  • Changed Requirements: None
  • Commit: aed7e8bb83f9ba165f7455b13d9e9586b3954956
  • Run: View logs

This comment is automatically updated per run

@danielsiegl danielsiegl marked this pull request as ready for review April 7, 2026 10:52
@danielsiegl danielsiegl merged commit 0784251 into main Apr 7, 2026
5 of 6 checks passed
@danielsiegl danielsiegl deleted the copilot/improve-polarion-display branch April 7, 2026 10:53
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.

Display when Polarion matches EA should be improved

2 participants