Skip to content

Comments

[CIT] Snapshots#23

Closed
arkjedrz wants to merge 3 commits intomainfrom
ajedrzejewski_cit_snapshots
Closed

[CIT] Snapshots#23
arkjedrz wants to merge 3 commits intomainfrom
ajedrzejewski_cit_snapshots

Conversation

@arkjedrz
Copy link

@arkjedrz arkjedrz commented Jul 7, 2025

Tests for snapshots.

@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: a6a43d5c-8809-4104-8305-39878166e947
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 (56 packages loaded, 10 targets configured)

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

Analyzing: target //:license-check (161 packages loaded, 997 targets configured)

Analyzing: target //:license-check (165 packages loaded, 2437 targets configured)

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

Analyzing: target //:license-check (171 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).
[9 / 13] checking cached actions
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.569s, Critical Path: 0.31s
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 force-pushed the ajedrzejewski_cit_snapshots branch from 737ffc8 to 451ffb4 Compare July 8, 2025 07:58
@arkjedrz arkjedrz force-pushed the ajedrzejewski_cit_snapshots branch from 451ffb4 to f282984 Compare July 11, 2025 09:05
@arkjedrz arkjedrz marked this pull request as ready for review July 11, 2025 09:06
@arkjedrz arkjedrz force-pushed the ajedrzejewski_cit_snapshots branch from f282984 to bce10de Compare July 11, 2025 09:56
@arkjedrz arkjedrz force-pushed the ajedrzejewski_cit_snapshots branch from bce10de to 1647998 Compare July 11, 2025 12:18
arkjedrz added 3 commits July 11, 2025 12:36
Applied automatic clippy fixes to decrease level of noisy warnings.
More flexible `get_value` implementation.
Previous version renamed to `get_value_as`.
Tests checking if snapshot behavior is correct.
@arkjedrz arkjedrz force-pushed the ajedrzejewski_cit_snapshots branch from 1647998 to b051c85 Compare July 11, 2025 13:10
@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.

1 participant