Skip to content

chore(bazel): use a minimal module declaration - #254

Open
AlexanderLanin wants to merge 1 commit into
mainfrom
repo-sync/bazel-module-one-line
Open

AlexanderLanin wants to merge 1 commit into
mainfrom
repo-sync/bazel-module-one-line

Conversation

@AlexanderLanin

@AlexanderLanin AlexanderLanin commented Aug 7, 2026 •

Copy link
Copy Markdown
Member

Policy

minimal-bazel-module-declaration

Keep the Bazel module declaration limited to its repository-owned name.

Why this repository?

This repository matches this policy because MODULE.bazel exists.

Changes

  • MODULE.bazel: replace matching text
    • Remove module metadata that should be derived by the Bazel Registry.

Tool revision

Generated from eclipse-score/tools at commit 3bc9906.


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.

@github-actions

github-actions Bot commented Aug 7, 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: d1bec218-4b51-4189-89dd-b57673e17fa8
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'score_docs_as_code', the root module requires module version score_docs_as_code@8.2.0, but got score_docs_as_code@8.3.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
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: 
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: 16.116s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@AlexanderLanin AlexanderLanin changed the title chore(bazel): make module declarations one line chore(bazel): use a minimal module declaration Aug 31, 2026
@eclipse-score-bot
eclipse-score-bot force-pushed the repo-sync/bazel-module-one-line branch from 2de1ec0 to f7578ab Compare September 22, 2026 09:04
@eclipse-score-bot
eclipse-score-bot force-pushed the repo-sync/bazel-module-one-line branch from f7578ab to 57cebd5 Compare September 30, 2026 01:22
@eclipse-score-bot
eclipse-score-bot force-pushed the repo-sync/bazel-module-one-line branch from 57cebd5 to da02061 Compare October 1, 2026 01:31
@eclipse-score-bot
eclipse-score-bot force-pushed the repo-sync/bazel-module-one-line branch from da02061 to 2436b2d Compare October 2, 2026 01:20
@eclipse-score-bot
eclipse-score-bot force-pushed the repo-sync/bazel-module-one-line branch from 2436b2d to 717f454 Compare October 2, 2026 17:26
@eclipse-score-bot
eclipse-score-bot force-pushed the repo-sync/bazel-module-one-line branch from 717f454 to b32247e Compare October 3, 2026 01:17
Copilot AI balanced review requested due to automatic review settings October 10, 2026 01:21
@eclipse-score-bot
eclipse-score-bot force-pushed the repo-sync/bazel-module-one-line branch from b32247e to cb0a967 Compare October 10, 2026 01:21

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused change correctly implements the stated minimal module declaration policy.

0 open findings

What changed in this PR

Simplifies the Bazel module declaration so registry-derived metadata is omitted.

Changes:

  • Retains only the repository-owned module name.
  • Removes the placeholder module version.
File Description
MODULE.bazel Reduces the module declaration to name only.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

This branch is waiting to be deployed

1 waiting deployment
workflow-approval — cb0a9672 Waiting Oct 10, 2026 by eclipse-score-bot via run-tests / coverage approval gate #1316
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

4 participants