Skip to content

Conversation

alexandruiulian10
Copy link
Contributor

@alexandruiulian10 alexandruiulian10 commented Mar 31, 2025

Provides a proposal for a safe JSON parser feature.

Copy link

github-actions bot commented Mar 31, 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: 62a09ee5-9071-436d-8e1f-6338083f0669
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
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/score/score/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/score/score/BUILD
INFO: Elapsed time: 4.964s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@alexandruiulian10 alexandruiulian10 force-pushed the json_feature_request branch 2 times, most recently from 2fb2df2 to 29b929c Compare April 1, 2025 12:37
Copy link

github-actions bot commented May 7, 2025

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

@alexandruiulian10 alexandruiulian10 force-pushed the json_feature_request branch 4 times, most recently from 5c9e3a9 to 70d0134 Compare May 9, 2025 13:58
Copy link
Contributor

@arsibo arsibo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@4og 4og self-requested a review July 29, 2025 15:37
4og
4og previously requested changes Jul 29, 2025
Copy link
Contributor

@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two comments originally from masc2023 still have to be resolved

Copy link
Contributor

@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see also updated inline comments

Copy link
Contributor

@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment that needed fixing resolved, so I approve

@aschemmel-tech aschemmel-tech dismissed 4og’s stale review August 8, 2025 09:54

The findings were resolved by later commits

@aschemmel-tech aschemmel-tech merged commit 23cebcb into eclipse-score:main Aug 11, 2025
6 checks passed
@alexandruiulian10 alexandruiulian10 deleted the json_feature_request branch September 9, 2025 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Feature request for qualified json-parser