Skip to content

Add bugfix for new module json_encoding quirks #42

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

MaximilianSoerenPollak
Copy link
Contributor

It seems that through upgrading some of the dependencies, the json_encoding inside bzl files is now different therefore the decoding we did in docs-as-code now doesn't work properly anymore and errors.

This PR fixes this error. Tested via proces repo with local path override.

Copy link

github-actions bot commented May 23, 2025

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //src:license-check

Status: ✅ Passed

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
INFO: Invocation ID: a39933c0-fddf-4293-8e5b-c528b5bfc2c0
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
Loading: 0 packages loaded
    currently loading: src
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)

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

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

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

Analyzing: target //src:license-check (144 packages loaded, 1738 targets configured)

Analyzing: target //src:license-check (151 packages loaded, 2633 targets configured)

Analyzing: target //src:license-check (156 packages loaded, 2668 targets configured)

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

INFO: Analyzed target //src:license-check (161 packages loaded, 4927 targets configured).
INFO: Found 1 target...
Target //src:license.check.license_check up-to-date:
  bazel-bin/src/license.check.license_check
  bazel-bin/src/license.check.license_check.jar
INFO: Elapsed time: 18.062s, Critical Path: 0.47s
INFO: 13 processes: 4 disk cache hit, 9 internal.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/src/license.check.license_check src/formatted.txt -review -project automotive.score -repo https://github.com/eclipse-score/docs-as-code -token otyhZ4eaRYK1tKLNNF-Y
[main] INFO Querying Eclipse Foundation for license data for 76 items.
[main] INFO Found 54 items.
[main] INFO Querying ClearlyDefined for license data for 22 items.
[main] INFO Found 22 items.
[main] INFO Vetted license information was found for all content. No further investigation is required.

@MaximilianSoerenPollak MaximilianSoerenPollak merged commit 2998a1e into eclipse-score:main May 26, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in Infrastructure May 26, 2025
@MaximilianSoerenPollak MaximilianSoerenPollak deleted the MSP_bugfix branch May 28, 2025 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants