Skip to content

Using new versions of docs-as-code - #189

Merged
rmaddikery merged 2 commits into
eclipse-score:mainfrom
etas-contrib:update-docs-as-code
Aug 18, 2026
Merged

Using new versions of docs-as-code#189
rmaddikery merged 2 commits into
eclipse-score:mainfrom
etas-contrib:update-docs-as-code

Conversation

@MaciejKaszynski

@MaciejKaszynski MaciejKaszynski commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Notes for Reviewer

Fixes #183
Applied the patch to remove dev_dependency from docs-as-code and platforms.

Note there are already 2 PRs:

Pre-Review Checklist for the PR Author

  • PR title is short, expressive and meaningful
  • Commits are properly organized
  • Relevant issues are linked in the References section
  • Tests are conducted
  • Unit tests are added

Checklist for the PR Reviewer

  • Commits are properly organized and messages are according to the guideline
  • Unit tests have been written for new behavior
  • Public API is documented
  • PR title describes the changes

Post-review Checklist for the PR Author

  • All open points are addressed and tracked via issues

References

Closes #

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //: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: e1b9de30-e6ba-4a38-96ac-1e1b36e4fb1c
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 4 packages loaded
Loading: 4 packages loaded
    currently loading: 
Loading: 4 packages loaded
    currently loading: 
Loading: 4 packages loaded
    currently loading: 
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/logging/logging/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/logging/logging/BUILD
INFO: Elapsed time: 10.660s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

Comment thread MODULE.bazel Outdated
@github-actions

Copy link
Copy Markdown

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

anmittag
anmittag previously approved these changes Jul 24, 2026
@anmittag

Copy link
Copy Markdown
Member

Meanwhile we shall use docs-as-code v7

Comment thread MODULE.bazel Outdated
@MaciejKaszynski
MaciejKaszynski deployed to workflow-approval August 18, 2026 12:12 — with GitHub Actions Active
@MaciejKaszynski
MaciejKaszynski deployed to workflow-approval August 18, 2026 12:12 — with GitHub Actions Active
@MaciejKaszynski
MaciejKaszynski deployed to workflow-approval August 18, 2026 12:12 — with GitHub Actions Active
@MaciejKaszynski
MaciejKaszynski deployed to workflow-approval August 18, 2026 12:12 — with GitHub Actions Active
@github-project-automation github-project-automation Bot moved this from Backlog to On Hold in LOG - Logging FT Aug 18, 2026
@rmaddikery
rmaddikery merged commit fd2aa58 into eclipse-score:main Aug 18, 2026
23 checks passed
@github-project-automation github-project-automation Bot moved this from On Hold to Done in LOG - Logging FT Aug 18, 2026
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.

Reference integration of logging fails due to transitive deps

3 participants