Skip to content

Conversation

@AlexanderLanin
Copy link
Member

Introduce a new decision record type and update the relevant functions to handle decision records in the Docs-As-Code workflows. This change enhances the documentation process by formalizing decision-making records.

@github-actions
Copy link

github-actions bot commented Aug 28, 2025

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //src:license-check

Status: ✅ Passed

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 142c6f15-637b-4def-b62b-b4b31795aa88
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //src:license-check (68 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (123 packages loaded, 2288 targets configured)

Analyzing: target //src:license-check (134 packages loaded, 2484 targets configured)

Analyzing: target //src:license-check (135 packages loaded, 2484 targets configured)

INFO: Analyzed target //src:license-check (137 packages loaded, 4500 targets configured).
INFO: Found 1 target...
Target //src:license.check.license_check up-to-date:
  bazel-bin/src/license.check.license_check
  bazel-bin/src/license.check.license_check.jar
[13 / 13] no actions running
INFO: Elapsed time: 13.744s, Critical Path: 0.40s
INFO: 13 processes: 4 disk cache hit, 9 internal.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/src/license.check.license_check src/formatted.txt <args omitted>
[main] INFO Querying Eclipse Foundation for license data for 83 items.
[main] INFO Found 58 items.
[main] INFO Querying ClearlyDefined for license data for 25 items.
[main] INFO Found 25 items.
[main] INFO Vetted license information was found for all content. No further investigation is required.

@github-actions
Copy link

github-actions bot commented Sep 8, 2025

The created documentation from the pull request is available at: docu-html

@AlexanderLanin AlexanderLanin marked this pull request as ready for review September 8, 2025 14:29
Copilot AI review requested due to automatic review settings September 8, 2025 14:29
Copy link

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

This PR introduces Decision Record support to formalize decision-making processes in the Docs-As-Code workflow. The implementation adds a new dec_rec need type to the metamodel with structured fields for context, decisions, status, and consequences.

  • Adds dec_rec need type to the metamodel with mandatory fields for ID, status, context, and decision
  • Updates ID validation to exempt decision records from feature-based location requirements
  • Migrates existing decision record from YAML frontmatter to the new structured format

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/extensions/score_metamodel/metamodel.yaml Defines the new dec_rec need type with required and optional fields
src/extensions/score_metamodel/checks/id_contains_feature.py Exempts decision records from feature-based ID validation
docs/internals/decisions_and_concepts/index.rst Adds a table to display decision records
docs/internals/decisions_and_concepts/001-test-results-in-workflow.md Converts existing decision record to new format

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@FScholPer FScholPer left a comment

Choose a reason for hiding this comment

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

looks good

@AlexanderLanin AlexanderLanin merged commit ecfc191 into main Sep 9, 2025
11 checks passed
@AlexanderLanin AlexanderLanin deleted the dr branch September 9, 2025 06:14
@github-project-automation github-project-automation bot moved this from Draft to Done in Infrastructure Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants