Skip to content
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

fix: CodeQL scan #509

Merged
merged 1 commit into from
Jan 30, 2025
Merged

fix: CodeQL scan #509

merged 1 commit into from
Jan 30, 2025

Conversation

d3xter666
Copy link
Contributor

CodeQL code scan complains about this file being broken. This is a case we test for the config manager. So, wrapping it in a JSON.parse to avoid the error there, but keep the test case valid.

@d3xter666 d3xter666 assigned d3xter666 and flovogt and unassigned d3xter666 and flovogt Jan 30, 2025
@d3xter666 d3xter666 requested review from a team and flovogt January 30, 2025 09:18
@d3xter666 d3xter666 merged commit 67e86d8 into main Jan 30, 2025
15 checks passed
@d3xter666 d3xter666 deleted the fix-codeql-scan branch January 30, 2025 11:04
@openui5bot openui5bot mentioned this pull request Jan 29, 2025
maxreichmann pushed a commit that referenced this pull request Jan 30, 2025
CodeQL code scan complains about this file being broken. This is a case
we test for the config manager. So, wrapping it in a JSON.parse to avoid
the error there, but keep the test case valid.
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.

2 participants