Skip to content

Add devcontainer, dependabot, some pre-commit hooks#48

Open
lurtz wants to merge 5 commits intoeclipse-score:mainfrom
elektrobit-contrib:add-devcontainer-dependabot-pre-commit-hooks
Open

Add devcontainer, dependabot, some pre-commit hooks#48
lurtz wants to merge 5 commits intoeclipse-score:mainfrom
elektrobit-contrib:add-devcontainer-dependabot-pre-commit-hooks

Conversation

@lurtz
Copy link
Contributor

@lurtz lurtz commented Mar 17, 2026

The module_template is the starting point of a new repo and should IMHO also define how an ideal repo looks like. Thus it should have a good out of the box experience for developers and automation.

The selected tools and configuration either make setup easier, prevent mistakes or just automate version management.

I will with another pull request also introduce the reuse tool via pre-commit hooks.

lurtz added 5 commits March 17, 2026 13:38
The devcontainer is a unified environment which should work in most cases without friction. This reduces setup time for developers.
Manually checking for updates is a pain in the ass and should be automated.
This prevents us to do easy to detect and fix mistakes
@github-actions
Copy link

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.3.0) and connecting to it...
INFO: Invocation ID: 05152780-4040-414e-89e9-bbc521823e78
Computing main repo mapping: 
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: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (17 packages loaded, 9 targets configured)

Analyzing: target //:license-check (77 packages loaded, 9 targets configured)

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

Analyzing: target //:license-check (84 packages loaded, 9 targets configured)

Analyzing: target //:license-check (145 packages loaded, 1006 targets configured)

Analyzing: target //:license-check (147 packages loaded, 2756 targets configured)

Analyzing: target //:license-check (152 packages loaded, 2804 targets configured)

Analyzing: target //:license-check (152 packages loaded, 2804 targets configured)

Analyzing: target //:license-check (152 packages loaded, 2804 targets configured)

Analyzing: target //:license-check (153 packages loaded, 2804 targets configured)

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

Analyzing: target //:license-check (156 packages loaded, 4931 targets configured)

Analyzing: target //:license-check (157 packages loaded, 5055 targets configured)

Analyzing: target //:license-check (157 packages loaded, 5055 targets configured)

Analyzing: target //:license-check (157 packages loaded, 5055 targets configured)

Analyzing: target //:license-check (157 packages loaded, 5055 targets configured)

Analyzing: target //:license-check (157 packages loaded, 5055 targets configured)

INFO: Analyzed target //:license-check (158 packages loaded, 6990 targets configured).
[7 / 13] 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)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 2 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[11 / 13] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar
[12 / 13] [Prepa] Building license.check.license_check.jar ()
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: 213.169s, Critical Path: 2.84s
INFO: 13 processes: 9 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 13 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
Copy link

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

@dcalavrezo-qorix
Copy link
Contributor

dcalavrezo-qorix commented Mar 17, 2026

what does the dependabot change in case of bazel?

have they added blzmod support? if so, I guess is fairly new
]would it bump versions in the MODULE.bazel ?

@lurtz
Copy link
Contributor Author

lurtz commented Mar 17, 2026

IIRC it bumps dependencies outside the S-CORE bazel registry. See e.g. https://github.com/eclipse-score/inc_someip_gateway/pull/46/changes

Lockfiles for e.g. devcontainer features are also bumped: https://github.com/eclipse-score/devcontainer/pull/89/changes

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.

2 participants