Skip to content

chore: process_description to 2.2.3 - #1025

Merged
AlexanderLanin merged 5 commits into
eclipse-score:mainfrom
etas-contrib:update-process-desc
Nov 5, 2026
Merged

AlexanderLanin merged 5 commits into
eclipse-score:mainfrom
etas-contrib:update-process-desc

Conversation

@a-zw

@a-zw a-zw commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

📌 Description

Update process_description to 2.2.3.

There is one minor change for a requirement which is not implemented yet. Upstream change was eclipse-score/process_description#853

🚨 Impact Analysis

  • This change does not violate any tool requirements and is covered by existing tool requirements
  • This change does not violate any design decisions
  • Otherwise I have created a ticket for new tool qualification

✅ Checklist

  • Added/updated documentation for new or changed features
  • Added/updated tests to cover the changes
  • Followed project coding standards and guidelines

@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-1025: https://eclipse-score.github.io/docs-as-code/pr-1025/

@a-zw
a-zw requested a balanced review from Copilot November 4, 2026 15:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The version-2 requirements omit repository/folder-specific level configuration and misclassify feature_integration_tests/ as Unit, which would reject valid Feature links.

1 open issue
What changed in this PR

Updates Docs-as-Code to process_description 2.2.3 and adjusts its verification requirements to reference the updated upstream specification.

Changes:

  • Bumps the Bazel dependency and corresponding lockfile entries to 2.2.3.
  • Updates the downstream integration-test fixture to use the same release.
  • Revises requirement traceability and the unimplemented test-link validation requirement.
File Description
tools/​tests/​module_verification_reports_test.py Updates the downstream fixture’s process_description version.
MODULE.bazel.lock Records registry hashes for version 2.2.3.
MODULE.bazel Bumps process_description from 2.2.2 to 2.2.3.
docs/​internals/​requirements/​requirements.rst References parent requirement version 2 and revises test-link rules.

Review effort: Balanced

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The metadata table conflicts with the 2.2.3 process guidance, potentially causing consumers to reject valid unit-level tags and accept unsupported explorative-testing level combinations.

No open issues 🎉

1 previously missed issue

In code that hasn't changed since last review

Low severity Align metadata table with updated test-level guidance

docs/​internals/​requirements/​requirements.rst:838

The metadata table below still contradicts the process guidance shipped in 2.2.3. Upstream PR #853 allows requirements-based and boundary-values at unit level, but restricts explorative-testing to feature/platform; this requirement excludes the former and permits the latter at all levels. Please align the table with the upgraded guidance so consumers implementing these checks do not reject valid metadata or accept unsupported level/tag combinations.

1 resolved since last review

Review effort: Balanced

@AlexanderLanin
AlexanderLanin merged commit 6d5a5ff into eclipse-score:main Nov 5, 2026
27 of 32 checks passed
@AlexanderLanin
AlexanderLanin deleted the update-process-desc branch November 5, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

4 participants