Skip to content

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

Merged
rmaddikery merged 1 commit into
mainfrom
repo-sync/bazel-module-one-line
Oct 27, 2026
Merged

rmaddikery merged 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 9025965.


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
Contributor

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: 75f610cc-6a40-442f-ba9f-47b426a80466
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'rules_cc', the root module requires module version rules_cc@0.2.17, but got rules_cc@0.2.25 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
DEBUG: /home/runner/.bazel/external/score_docs_as_code+/docs.bzl:716:14: INFO: ⚠️ DEPRECATED: Passing a Needs inventory through docs(data = [...]) is deprecated!!! Move it to external_needs = [...] instead.
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: 15.626s
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.

@eclipse-score-bot
eclipse-score-bot force-pushed the repo-sync/bazel-module-one-line branch from cb0a967 to fc2ed3e Compare October 13, 2026 01:24

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 declaration remains valid, preserves module identity, and has no identified blocking issues.

0 open findings

🧠 Review effort: Balanced

@hoppe-and-dreams
hoppe-and-dreams force-pushed the repo-sync/bazel-module-one-line branch from 4ad143e to 9f3d94e Compare October 20, 2026 09:51
Copilot AI balanced review requested due to automatic review settings October 20, 2026 09:51

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 narrowly scoped change uses valid Bazel syntax, preserves module identity, and has no identified issues.

No open issues 🎉

Review effort: Balanced

@rmaddikery
rmaddikery force-pushed the repo-sync/bazel-module-one-line branch from 9f3d94e to e9cf66e Compare October 27, 2026 10:44
Copilot AI balanced review requested due to automatic review settings October 27, 2026 10:44
@AlexanderLanin
AlexanderLanin deployed to workflow-approval October 27, 2026 10:44 — with GitHub Actions Active
@rmaddikery
rmaddikery deployed to workflow-approval October 27, 2026 10:44 — with GitHub Actions Active

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 declaration remains valid, matches the stated policy, and no dependency on the removed placeholder version was identified.

No open issues 🎉

Review effort: Balanced

@rmaddikery
rmaddikery force-pushed the repo-sync/bazel-module-one-line branch from e9cf66e to 002f88d Compare October 27, 2026 10:53
Copilot AI balanced review requested due to automatic review settings October 27, 2026 10:53
@rmaddikery
rmaddikery temporarily deployed to workflow-approval October 27, 2026 10:53 — with GitHub Actions Inactive

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 narrow metadata and lockfile changes match the stated policy and introduce no identified blocking issues.

No open issues 🎉

Review effort: Balanced

@github-actions

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-254: https://eclipse-score.github.io/logging/pr-254/

@github-project-automation github-project-automation Bot moved this from Backlog to On Hold in LOG - Logging FT Oct 27, 2026
@rmaddikery
rmaddikery merged commit b87b5d3 into main Oct 27, 2026
22 checks passed
@rmaddikery
rmaddikery deleted the repo-sync/bazel-module-one-line branch October 27, 2026 12:21
@github-project-automation github-project-automation Bot moved this from On Hold to Done in LOG - Logging FT Oct 27, 2026

This branch was previously deployed

1 inactive deployment
workflow-approval — 002f88d6 Deployed Oct 27, 2026 by rmaddikery via run-tests / coverage approval gate #1445
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.

5 participants