Repository navigation
chore(bazel): align development-only dependencies - #818
Merged
Merged
Conversation
eclipse-score-bot
requested review from
AlexanderLanin and
MaximilianSoerenPollak
as code owners
September 12, 2026 01:04
eclipse-score-bot
requested review from
a-zw,
dcalavrezo-qorix and
nradakovic
as code owners
September 12, 2026 01:04
a-zw
reviewed
Sep 15, 2026
| # 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) |
Contributor
There was a problem hiding this comment.
Why does it adapt aspect_rules_py? The description only mentions buildifier_prebuilt.
Contributor
There was a problem hiding this comment.
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?
eclipse-score-bot
force-pushed
the
repo-policy-sync/dev-dependency-alignment
branch
from
September 22, 2026 08:58
a5ab0c5 to
5d1ca70
Compare
Contributor
|
Documentation preview for this pull request is available at: |
AlexanderLanin
approved these changes
Sep 22, 2026
AlexanderLanin
deleted the
repo-policy-sync/dev-dependency-alignment
branch
September 22, 2026 09:48
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Policy
dev-dependency-alignmentKeep 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.bazeldeclares 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 trueTool 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.