Skip to content

testing: C++ CIT - default values#208

Merged
PiotrKorkus merged 2 commits intoeclipse-score:mainfrom
qorix-group:arkjedrz_cpp-default-values-cit-reworked
Jan 14, 2026
Merged

testing: C++ CIT - default values#208
PiotrKorkus merged 2 commits intoeclipse-score:mainfrom
qorix-group:arkjedrz_cpp-default-values-cit-reworked

Conversation

@arkjedrz
Copy link
Contributor

Default values tests for C++ impl.

@arkjedrz arkjedrz requested a review from PiotrKorkus January 11, 2026 13:31
@github-actions
Copy link

github-actions bot commented Jan 11, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //: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: 91b89cbf-d8e4-41a1-9ddf-cbb047bea20b
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
DEBUG: Rule 'rust_qnx8_toolchain+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-oEubHgeZDdT0svMmBKJx7c3/2TdSI/vfwRUyDn+TPGA="
DEBUG: Repository rust_qnx8_toolchain+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:394:31: in <toplevel>
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'score_process', the root module requires module version score_process@1.3.2, but got score_process@1.4.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Loading: 
Loading: 3 packages loaded
Loading: 3 packages loaded
    currently loading: 
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)

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

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

Analyzing: target //:license-check (136 packages loaded, 468 targets configured)

Analyzing: target //:license-check (158 packages loaded, 6006 targets configured)

Analyzing: target //:license-check (159 packages loaded, 6997 targets configured)

INFO: Analyzed target //:license-check (162 packages loaded, 9013 targets configured).
[9 / 14] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 62 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
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: 24.646s, Critical Path: 0.44s
INFO: 14 processes: 5 disk cache hit, 9 internal.
INFO: Build completed successfully, 14 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

@arkjedrz arkjedrz force-pushed the arkjedrz_cpp-default-values-cit-reworked branch 2 times, most recently from 78e59b9 to c0a1b63 Compare January 11, 2026 13:52
@PiotrKorkus PiotrKorkus requested a review from Saumya-R January 12, 2026 07:31
@arkjedrz arkjedrz force-pushed the arkjedrz_cpp-default-values-cit-reworked branch from c0a1b63 to 2cb6d35 Compare January 12, 2026 08:26
@arkjedrz arkjedrz self-assigned this Jan 12, 2026
@arkjedrz arkjedrz force-pushed the arkjedrz_cpp-default-values-cit-reworked branch from 2cb6d35 to 4895c2f Compare January 12, 2026 12:27
Saumya-R and others added 2 commits January 12, 2026 14:40
- Added C++ test scenarios for default values testing
- default_values, remove_key, reset operations, checksum
- Refactored and reworked implementation.
- No longer requires SIGABRT for some cases.
@arkjedrz arkjedrz force-pushed the arkjedrz_cpp-default-values-cit-reworked branch from 4895c2f to 1f5b616 Compare January 12, 2026 14:40
Copy link
Contributor

@PiotrKorkus PiotrKorkus left a comment

Choose a reason for hiding this comment

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

lgtm

@PiotrKorkus PiotrKorkus merged commit 69cb401 into eclipse-score:main Jan 14, 2026
14 checks passed
@arkjedrz arkjedrz deleted the arkjedrz_cpp-default-values-cit-reworked branch January 14, 2026 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants