Skip to content

Conversation

@NilsEb
Copy link
Collaborator

@NilsEb NilsEb commented Jul 16, 2025

  • Remove the license check workflow from s-core, which is mainly intended for rust and python (see eclipse dash license tool)
  • Add the workflow ci-fuzz from nlohmann_json which runs 5 minutes of fuzz testing.

Notably, this is not related to the continuous Google OSS fuzz testing that is used for nlohmann_json (see dashboard and configuration). The relevance threshold to be considered for Google OSS fuzz testing is probably not yet met. Nevertheless, the results from Google OSS fuzz testing of nlohmann_json may be considered as evidence.

@github-actions
Copy link

github-actions bot commented Jul 16, 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 (8.3.1) and connecting to it...
INFO: Invocation ID: 778098d2-8cf2-4f1b-8a0f-cdbc72b15e4a
Computing main repo mapping: 
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.14.0.bcr.1 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'score_python_basics', the root module requires module version score_python_basics@0.3.0, but got score_python_basics@0.3.2 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Loading: 
Loading: 0 packages loaded
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/json/json/BUILD.bazel
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/json/json/BUILD.bazel
INFO: Elapsed time: 5.796s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@github-actions
Copy link

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

@coveralls
Copy link

coveralls commented Jul 16, 2025

Coverage Status

coverage: 99.186%. remained the same
when pulling d34e286 on add_cifuzz_workflow
into 25b4cc4 on main.

@NilsEb NilsEb requested a review from FelixMoelders July 16, 2025 13:15
@NilsEb NilsEb changed the title Add ci-fuzz workflow Add ci-fuzz workflow and remove license check Jul 18, 2025
@NilsEb NilsEb merged commit 415501a into main Jul 18, 2025
72 checks passed
@NilsEb NilsEb deleted the add_cifuzz_workflow branch July 18, 2025 13:43
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.

4 participants