eclipse-score / eclipse-score/score

Component Requirements Inspection - Launch Manager and Health Monitor

Open
#3,035 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Starlark
Stars
109
Forks
105
Avg merge
22h 15m
Merged PRs (30d)
22

Description

### What

This issue tracks the formal Component Requirements Inspection for the component specified above, according to the S-CORE process.

The inspection validates that the component's requirements meet quality, completeness, and testability criteria before the work product is marked as `valid(inspected)`.

References:
- [S-CORE Inspection Conduct](https://eclipse-score.github.io/process_description/main/general_concepts/score_review_concept.html#inspection-conduct)
- [Inspection Checklist Template](https://eclipse-score.github.io/module_template/main/score/component_example/docs/requirements/chklst_req_inspection.html)

### Acceptance Criteria (DoD)

- [ ] Component Requirements in valid(inspected) state
- [ ] Inspection Checklist filled and merged

### How

#### Moderator (Safety Manager)
- [ ] Create the inspection PR in `eclipse-score/lifecycle` with the inspection checklist and link it to this issue via `Fixes #` in the PR description
- [ ] Assign Reviewer(s) and Test Expert; communicate scope and deadline
- [ ] Incorporate reviewer findings as commits into the inspection PR
- [ ] Ensure all checklist items are documented with pass/fail verdicts
- [ ] After all reviewers approve, merge the PR
- [ ] Set work product status to `valid(inspected)`

#### Reviewer(s)
- [ ] Review all component requirements against each inspection checklist item
- [ ] For each failing item, open a GitHub issue in `eclipse-score/lifecycle` to track the required correction; link it in the PR comment
- [ ] Add findings as PR comments (pass or fail, with rationale and link to correction issue where applicable)
- [ ] Set PR to "Request Changes" if findings exist
- [ ] Re-review after the author addresses findings; update verdict accordingly

#### Test Expert
- [ ] Review requirements from a testability perspective (focus: REQ_08_01 checklist item)
- [ ] Verify that test cases can be derived from each requirement and that verification criteria are clear
- [ ] Add testability findings as PR comments
- [ ] Re-review after author addresses findings; update verdict accordingly

#### Content Responsible (Author)
- [ ] Analyze all findings and linked correction issues documented in the PR
- [ ] Respond to each finding with an explanation or a proposed correction
- [ ] Commit corrections to address all findings
- [ ] Request re-review from the Reviewer(s) and Test Expert
- [ ] Address any follow-up comments until all reviewers approve

> **Note:** Eclipse contributors cannot push to the inspection branch directly. Findings must be added as PR comments; the Moderator incorporates them as commits.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the S-CORE Inspection Conduct reference and the Inspection Checklist Template linked in the issue. Review the component requirements against the checklist, then create an inspection PR in eclipse-score/lifecycle with documented pass/fail findings and link any correction issues. Done means the checklist is filled and merged and the component reaches valid(inspected) status.

Written by the indexing model from the issue text.

Assessment

Tech stack
github
Domain
documentation
Issue type
Documentation
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.