Skip to content

chore(bazel): align development-only dependencies - #1072

Closed
eclipse-score-bot wants to merge 1 commit into
mainfrom
repo-policy-sync/dev-dependency-alignment
Closed

eclipse-score-bot wants to merge 1 commit into
mainfrom
repo-policy-sync/dev-dependency-alignment

Conversation

@eclipse-score-bot

@eclipse-score-bot eclipse-score-bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Policy

dev-dependency-alignment

Keep Bazel toolchain, lint, image, and documentation dependencies in the
intended dependency graph. The listed build-support dependencies are
development-only, while score_docs_as_code is used by the normal build graph.

Why this repository?

This repository matches this policy because MODULE.bazel declares at least one of these direct Bazel dependencies: score_bazel_cpp_toolchains, buildifier_prebuilt, aspect_rules_lint, toolchains_llvm, score_rules_imagefs, score_docs_as_code.

Changes

  • MODULE.bazel: remove dev_dependency from Bazel dependency 'score_docs_as_code'
    • Documentation rules are required by the normal build graph.

Tool revision

Generated from eclipse-score/tools at commit 5934dc0.


This pull request is managed by SCORE Repository Policy Sync and may be updated by a later policy run.
Please report any issues to #score-infrastructure.

Note

This pull request is generated automatically. Review the proposed changes
before merging.

@eclipse-score-bot
eclipse-score-bot force-pushed the repo-policy-sync/dev-dependency-alignment branch from d682bb0 to 6d91d1a Compare September 22, 2026 08:57

@limdor limdor 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.

I do not see why this is not a dev dependency. In fact is not even a direct dependency, just a transitive one. It is transitive through baselibs and baselibs declares it as none dev.
To me it looks like the logic to deduce if it is a dev dependency or no is broken, it should only consider the code of this module and not the code of other modules.

@limdor

limdor commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

The plan is to delete it because it is not even used #1136

@limdor limdor closed this Sep 22, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in COM - Communication FT Sep 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants