Skip to content

Comments

Add non-generic get_value implementation#19

Closed
arkjedrz wants to merge 2 commits intomainfrom
ajedrzejewski_nongeneric_get_value
Closed

Add non-generic get_value implementation#19
arkjedrz wants to merge 2 commits intomainfrom
ajedrzejewski_nongeneric_get_value

Conversation

@arkjedrz
Copy link

@arkjedrz arkjedrz commented Jul 7, 2025

Resolve #12

@arkjedrz arkjedrz self-assigned this Jul 7, 2025
@github-actions
Copy link

github-actions bot commented Jul 7, 2025

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 and connecting to it...
INFO: Invocation ID: 990d6605-d213-4728-8232-1305d16c443f
Computing main repo mapping: 
Computing main repo mapping: 
DEBUG: Rule 'rules_boost~' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-aSViuHoOdF32Ue/Cx02/mOzyEebHMFuifjZs4GNWjsE="
DEBUG: Repository rules_boost~ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:387:31: in <toplevel>
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'googletest', the root module requires module version googletest@1.14.0, but got googletest@1.15.0 in the resolved dependency graph.
WARNING: For repository 'aspect_rules_lint', the root module requires module version aspect_rules_lint@1.0.3, but got aspect_rules_lint@1.4.2 in the resolved dependency graph.
Computing main repo mapping: 
Loading: 
Loading: 1 packages loaded
Analyzing: target //:license-check (2 packages loaded, 0 targets configured)
Analyzing: target //:license-check (2 packages loaded, 0 targets configured)

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

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

Analyzing: target //:license-check (162 packages loaded, 1916 targets configured)

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

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

Analyzing: target //:license-check (174 packages loaded, 5025 targets configured)

Analyzing: target //:license-check (175 packages loaded, 5145 targets configured)

Analyzing: target //:license-check (175 packages loaded, 5145 targets configured)

INFO: Analyzed target //:license-check (176 packages loaded, 7064 targets configured).
[8 / 13] Creating runfiles tree bazel-out/k8-opt-exec-ST-d57f47055a04/bin/external/score_dash_license_checker~/tool/formatters/dash_format_converter.runfiles [for tool]; 0s local
ERROR: /home/runner/work/inc_mw_per/inc_mw_per/BUILD:30:21: Generating Dash formatted dependency file ... failed: missing input file '//:cargo_lock'
ERROR: /home/runner/work/inc_mw_per/inc_mw_per/BUILD:30:21: Generating Dash formatted dependency file ... failed: 1 input file(s) do not exist
Target //:license.check.license_check failed to build
Use --verbose_failures to see the command lines of failed build steps.
ERROR: /home/runner/work/inc_mw_per/inc_mw_per/BUILD:30:21 Generating Dash formatted dependency file ... failed: 1 input file(s) do not exist
INFO: Elapsed time: 184.714s, Critical Path: 0.37s
INFO: 10 processes: 10 internal.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@github-actions
Copy link

github-actions bot commented Jul 7, 2025

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

@arkjedrz arkjedrz marked this pull request as ready for review July 7, 2025 10:05
@arkjedrz arkjedrz force-pushed the ajedrzejewski_nongeneric_get_value branch 3 times, most recently from 2e5010d to 4858515 Compare July 11, 2025 09:19
@arkjedrz arkjedrz force-pushed the ajedrzejewski_nongeneric_get_value branch from 4858515 to ee9e452 Compare July 11, 2025 12:17
Applied automatic clippy fixes to decrease level of noisy warnings.
@arkjedrz arkjedrz force-pushed the ajedrzejewski_nongeneric_get_value branch from ee9e452 to 1547525 Compare July 11, 2025 12:47
More flexible `get_value` implementation.
Previous version renamed to `get_value_as`.
@arkjedrz arkjedrz force-pushed the ajedrzejewski_nongeneric_get_value branch from 1547525 to 97034c3 Compare July 11, 2025 12:57
@arkjedrz arkjedrz closed this Jul 11, 2025
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.

Add non-generic get_value implementation

1 participant