Skip to content

update get_value function - #417

Merged
umaucher merged 3 commits into
eclipse-score:mainfrom
Ahmed-Elsaka-JC:afe_APIS_update
Sep 18, 2026
Merged

umaucher merged 3 commits into
eclipse-score:mainfrom
Ahmed-Elsaka-JC:afe_APIS_update

Conversation

@Ahmed-Elsaka-JC

Copy link
Copy Markdown
Contributor
  • update get_value to return error if value not exist

@Ahmed-Elsaka-JC
Ahmed-Elsaka-JC deployed to workflow-approval September 10, 2026 16:22 — with GitHub Actions Active
@Ahmed-Elsaka-JC
Ahmed-Elsaka-JC deployed to workflow-approval September 10, 2026 16:22 — with GitHub Actions Active
@github-actions

github-actions Bot commented Sep 10, 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: 76493dd3-4abf-433c-91d5-1f6b35309802
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: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 3 packages loaded
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)
Analyzing: target //:license-check (4 packages loaded, 0 targets configured)

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

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

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

Analyzing: target //:license-check (150 packages loaded, 3134 targets configured)

Analyzing: target //:license-check (150 packages loaded, 3134 targets configured)

Analyzing: target //:license-check (150 packages loaded, 3134 targets configured)

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

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

Analyzing: target //:license-check (170 packages loaded, 9367 targets configured)

Analyzing: target //:license-check (173 packages loaded, 9399 targets configured)

Analyzing: target //:license-check (173 packages loaded, 9399 targets configured)

Analyzing: target //:license-check (176 packages loaded, 11287 targets configured)

Analyzing: target //:license-check (177 packages loaded, 11411 targets configured)

INFO: Analyzed target //:license-check (178 packages loaded, 11537 targets configured).
[13 / 17] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions, 1 running)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 66 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[15 / 17] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.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: 35.339s, Critical Path: 2.96s
INFO: 17 processes: 12 internal, 4 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 17 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
Copy Markdown
Contributor

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

@Ahmed-Elsaka-JC
Ahmed-Elsaka-JC marked this pull request as ready for review September 14, 2026 09:57
Comment thread score/kvs/rust_kvs/src/kvs.rs Outdated
Comment thread score/kvs/rust_kvs/src/kvs.rs Outdated
- update get_value to return error if value not exist
- remove fallback to default for get_value_as function
- update unit test for kvs_cpp
@umaucher
umaucher added this pull request to the merge queue Sep 18, 2026
Merged via the queue into eclipse-score:main with commit 6a29625 Sep 18, 2026
27 of 28 checks passed

This branch was successfully deployed

1 active deployment
workflow-approval — a5ac4e39 Deployed Sep 18, 2026 by Ahmed-Elsaka-JC via qnx-build (per-arm64-qnx) / approval #382
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