Skip to content

Conversation

@AlexanderLanin
Copy link
Member

Old attempt of semi-architecture. But we dont want to do architecture for docs-as-code tool.

@github-actions
Copy link

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //src:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: fd4a78e2-7e25-4141-8d9d-80b3cd9b6c02
Computing main repo mapping: 
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 (72 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (126 packages loaded, 2309 targets configured)

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

INFO: Analyzed target //src:license-check (137 packages loaded, 4500 targets configured).
[3 / 12] [Prepa] Creating source manifest for @@score_tooling+//dash/tool/formatters:dash_format_converter [for tool]
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
INFO: Elapsed time: 14.566s, Critical Path: 0.38s
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>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@RolandJentschETAS
Copy link
Contributor

This PR fixes #354.

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@MaximilianSoerenPollak MaximilianSoerenPollak dismissed their stale review January 21, 2026 09:24

Did not see that tests don't pass.

@MaximilianSoerenPollak
Copy link
Contributor

There is testcases now linked to something that doesn't exist.

Need 'testcase__test_strips_whitespace_EWKFN' has unknown outgoing link 'tool_req__docs_metamodel' in field 'partially_verifies' [needs.external_link_outgoing]
WARNING: https://github.com/etas-contrib/score_docs-as-code/blob/ae8faa972c6ecdd9f22becb688205857c8a7e5fa/src/extensions/score_metamodel/tests/test_metamodel__init__.py#L75: Need 'testcase__test_raises_assertion_for_invalid_check_PVYCN' has unknown outgoing link 'tool_req__docs_metamodel' in field 'partially_verifies' [needs.external_link_outgoing]

We probably should fix that?

@AlexanderLanin
Copy link
Member Author

Hups

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Draft

Development

Successfully merging this pull request may close these issues.

4 participants