Skip to content

Comply with updated design-documentation, reqstream-usage, and reviewmark-usage standards#153

Merged
Malcolmnixon merged 3 commits intomainfrom
copilot/update-repository-to-comply-with-standards
Apr 2, 2026
Merged

Comply with updated design-documentation, reqstream-usage, and reviewmark-usage standards#153
Malcolmnixon merged 3 commits intomainfrom
copilot/update-repository-to-comply-with-standards

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 2, 2026

  • Rename docs/reqstream/ files to remove unit-/subsystem- prefixes and restructure ots/ subfolder
  • Update requirements.yaml includes to reference new file paths
  • Update .reviewmark.yaml paths; remove redundant ReqStream-OTS review-set
  • Fix yamllint empty-lines error in .reviewmark.yaml (trailing blank line after removing OTS section)
  • Update docs/design/introduction.md to remove Linting subsystem references

…ge standards

Agent-Logs-Url: https://github.com/demaconsulting/ReqStream/sessions/b1d5559d-2881-41c5-bfc1-9eb8e83a8aea

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
…quirements)

Agent-Logs-Url: https://github.com/demaconsulting/ReqStream/sessions/00b814e4-e460-4933-95c5-76c0f36ced47

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI requested a review from Malcolmnixon April 2, 2026 13:17
@Malcolmnixon Malcolmnixon marked this pull request as ready for review April 2, 2026 13:21
Copilot AI review requested due to automatic review settings April 2, 2026 13:21
@Malcolmnixon
Copy link
Copy Markdown
Member

@copilot Please fix the issues reported by linting

Agent-Logs-Url: https://github.com/demaconsulting/ReqStream/sessions/576423a7-a19e-4a8e-880f-074654a1f509

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 2, 2026

@copilot Please fix the issues reported by linting

Fixed in 335442b — the yamllint empty-lines error in .reviewmark.yaml was a trailing blank line left behind when removing the OTS review-set.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://storage.googleapis.com/chromium-browser-snapshots/Linux_x64/1108766/chrome-linux.zip
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node install.js (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Aligns ReqStream’s requirements/docs structure and ReviewMark configuration with updated .github/standards, including standardized requirement file naming/layout and updated review-set coverage.

Changes:

  • Renames and reorganizes docs/reqstream/ requirement YAML files (including moving OTS requirements into docs/reqstream/ots/) and updates root requirements.yaml includes.
  • Updates .reviewmark.yaml to include requirements.yaml in needs-review and to point review-sets at the renamed requirement files.
  • Updates docs/design/introduction.md to remove the stale Linting subsystem references and reflect the current source layout.

Reviewed changes

Copilot reviewed 3 out of 19 changed files in this pull request and generated no comments.

Show a summary per file
File Description
requirements.yaml Updates includes: to reference renamed/moved requirement YAML files.
.reviewmark.yaml Updates review coverage patterns and review-set paths to match new docs/reqstream layout.
docs/design/introduction.md Removes stale Linting subsystem references and updates source layout diagram.
docs/reqstream/system.yaml Adds system-level requirements under the new naming convention.
docs/reqstream/program.yaml Adds program-unit requirements under the new naming convention.
docs/reqstream/cli/cli.yaml Adds CLI subsystem requirements under the new naming convention.
docs/reqstream/cli/context.yaml Adds Context unit requirements under the new naming convention.
docs/reqstream/modeling/modeling.yaml Adds Modeling subsystem requirements under the new naming convention.
docs/reqstream/modeling/requirements.yaml Adds Modeling unit + lint/loading requirements under the new naming convention.
docs/reqstream/tracing/tracing.yaml Adds Tracing subsystem requirements under the new naming convention.
docs/reqstream/tracing/trace-matrix.yaml Adds TraceMatrix unit requirements under the new naming convention.
docs/reqstream/self-test/self-test.yaml Adds SelfTest subsystem requirements under the new naming convention.
docs/reqstream/self-test/validation.yaml Adds Validation unit requirements under the new naming convention.
docs/reqstream/ots/mstest.yaml Moves MSTest OTS requirements into ots/ structure.
docs/reqstream/ots/buildmark.yaml Moves BuildMark OTS requirements into ots/ structure.
docs/reqstream/ots/versionmark.yaml Moves VersionMark OTS requirements into ots/ structure.
docs/reqstream/ots/sarifmark.yaml Moves SarifMark OTS requirements into ots/ structure.
docs/reqstream/ots/sonarmark.yaml Moves SonarMark OTS requirements into ots/ structure.
docs/reqstream/ots/reviewmark.yaml Moves ReviewMark OTS requirements into ots/ structure.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

3 participants