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: red-cross in commit status #7

Merged
merged 1 commit into from
Dec 20, 2023
Merged

fix: red-cross in commit status #7

merged 1 commit into from
Dec 20, 2023

Conversation

LecrisUT
Copy link
Owner

@LecrisUT LecrisUT commented Dec 20, 2023

#6 did not mask the results properly, this one should do the trick.

GitHub is not consistent when a variable is converted to a string and when it is a json-native object. In this case, the matrix variable is kept as json-native bool

Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@LecrisUT LecrisUT merged commit b994a4d into main Dec 20, 2023
13 of 22 checks passed
@LecrisUT LecrisUT deleted the ci/red-cross branch December 20, 2023 16:15
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