Skip to content

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

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

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: set Bazel dependency 'buildifier_prebuilt' dev_dependency to true
    • Buildifier is a development-time formatting tool.

Tool revision

Generated from eclipse-score/tools at commit b985606.


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.

Comment thread MODULE.bazel
# Additional Python rules provided by aspect, e.g. an improved version of
bazel_dep(name = "aspect_rules_py", version = "1.4.0")
bazel_dep(name = "buildifier_prebuilt", version = "8.2.0.2")
bazel_dep(name = "aspect_rules_py", version = "1.4.0", dev_dependency = True)

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.

Why does it adapt aspect_rules_py? The description only mentions buildifier_prebuilt.

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.

It should not touch aspect_rules_py.

I think that is wrong, as there is another policy that deletes the dev_dependency or sets it to False as far as I know.

@AlexanderLanin I'm remembering right, right?

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

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants