eclipse-score / eclipse-score/score
Component Requirements Inspection - Launch Manager and Health Monitor
- 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