Skip to content

Push .bazelversion to all dependables (#244)#85

Open
Subramanian-K812 wants to merge 5 commits into
eclipse-score:mainfrom
qorix-group:Subramanian-K812_fix_bazelversion_sync
Open

Push .bazelversion to all dependables (#244)#85
Subramanian-K812 wants to merge 5 commits into
eclipse-score:mainfrom
qorix-group:Subramanian-K812_fix_bazelversion_sync

Conversation

@Subramanian-K812

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Jun 4, 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: 846cd64d-ecc3-485b-9247-4f6e7d2db4e4
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (36 packages loaded, 10 targets configured)

Analyzing: target //:license-check (78 packages loaded, 10 targets configured)

Analyzing: target //:license-check (86 packages loaded, 10 targets configured)

Analyzing: target //:license-check (112 packages loaded, 418 targets configured)

Analyzing: target //:license-check (155 packages loaded, 4157 targets configured)

Analyzing: target //:license-check (160 packages loaded, 5468 targets configured)

Analyzing: target //:license-check (160 packages loaded, 5468 targets configured)

Analyzing: target //:license-check (160 packages loaded, 5468 targets configured)

Analyzing: target //:license-check (160 packages loaded, 5468 targets configured)

Analyzing: target //:license-check (163 packages loaded, 7353 targets configured)

Analyzing: target //:license-check (166 packages loaded, 10152 targets configured)

Analyzing: target //:license-check (167 packages loaded, 10160 targets configured)

Analyzing: target //:license-check (167 packages loaded, 10160 targets configured)

Analyzing: target //:license-check (167 packages loaded, 10160 targets configured)

Analyzing: target //:license-check (168 packages loaded, 10280 targets configured)

Analyzing: target //:license-check (168 packages loaded, 10280 targets configured)

Analyzing: target //:license-check (168 packages loaded, 10280 targets configured)

INFO: Analyzed target //:license-check (169 packages loaded, 11859 targets configured).
[9 / 16] Creating runfiles tree bazel-out/k8-opt-exec-ST-d57f47055a04/bin/external/score_tooling+/dash/tool/formatters/dash_format_converter.runfiles [for tool]; 0s local ... (2 actions, 1 running)
[12 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions running)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 2 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 205.047s, Critical Path: 2.63s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@Subramanian-K812

Copy link
Copy Markdown
Contributor Author

fix sync workflow to push .bazelversion to all "dependable elements"
Closes #85

@Subramanian-K812 Subramanian-K812 marked this pull request as ready for review June 4, 2026 08:38

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

please fix gitlint

Comment thread .github/workflows/sync-template.yml Outdated
Comment on lines +16 to +18
push:
branches: [main]
paths: [".bazelversion"]

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.

not needed? we can leave it as only scheduled?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The push trigger is intentional—it provides immediate sync when .bazelversion changes, ensuring dependent repos stay current without waiting for the monthly schedule. But to not complicate it further I can remove it.

@Subramanian-K812 Subramanian-K812 force-pushed the Subramanian-K812_fix_bazelversion_sync branch from 18edaa9 to 3da6f50 Compare June 5, 2026 03:53
Comment thread index.rst

@AlexanderLanin AlexanderLanin Jun 5, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why is this in the PR? Some merge problem?

@AlexanderLanin

AlexanderLanin commented Jun 5, 2026

Copy link
Copy Markdown
Member

This workflow is supposed to create PRs in other repos? Which repos? And can we try that with one repo first?

No, wait, sorry. If this workflow creates PRs in other repos, it should not be in the template, as everyone will have a copy of it! 😆
Or is it designed as pull? So it's not supposed to run in module_template at all.

@AlexanderLanin

AlexanderLanin commented Jun 5, 2026

Copy link
Copy Markdown
Member

Suggestion: add a job to https://github.com/eclipse-score/cicd-workflows/blob/main/.github/workflows/_local_daily.yml, something like "module_template_rollout". Which will create PRs in all other repositories that have a .bazelversion file.

@Subramanian-K812

Copy link
Copy Markdown
Contributor Author

@AlexanderLanin created PR for the above change. Please find it here.
eclipse-score/cicd-workflows#119

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

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

5 participants