Skip to content

fix: docs data handling and improve bundle documentation - #751

Merged
MaximilianSoerenPollak merged 2 commits into
eclipse-score:mainfrom
etas-contrib:bundle-docs
Aug 26, 2026
Merged

MaximilianSoerenPollak merged 2 commits into
eclipse-score:mainfrom
etas-contrib:bundle-docs

Conversation

@AlexanderLanin

Copy link
Copy Markdown
Member

No description provided.

@github-actions

github-actions Bot commented Aug 25, 2026 •

Copy link
Copy Markdown
Contributor

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //src:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 30fb6a8a-7135-4ae1-9380-88d2400b8c03
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
WARNING: Target pattern parsing failed.
ERROR: Skipping '//src:license-check': no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
ERROR: no such target '//src:license-check': target 'license-check' not declared in package 'src' defined by /home/runner/work/docs-as-code/docs-as-code/src/BUILD
INFO: Elapsed time: 5.978s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@github-actions

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-751: https://eclipse-score.github.io/docs-as-code/pr-751/

Comment thread bzl/bundle_rules.bzl
Comment on lines +160 to +162
# Encode the complete label identity so every target gets a stable,
# distinct synthetic path. In particular, local and external labels remain
# distinct when both are composed by one consuming project.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Would be nice (for future us) to add input => output

@MaximilianSoerenPollak MaximilianSoerenPollak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looked through & talked about in a call. Looks good

@MaximilianSoerenPollak
MaximilianSoerenPollak merged commit 88315f7 into eclipse-score:main Aug 26, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants