About PR 5916 : Sarif-PHP-Converters 1.3.0 #5939
Closed
llaville
started this conversation in
Show and tell
Replies: 3 comments
-
Theres an issue with PRs noted in GitHub status, started like less than 30 mins ago |
Beta Was this translation helpful? Give feedback.
0 replies
-
GH PR system is now operational. |
Beta Was this translation helpful? Give feedback.
0 replies
-
I've just finished to apply fix to my package and test it on MegaLinter context. My tests at least with PHPSTAN PASSED as expected
I'll release v1.3.1 in next minutes now. @echoix I'll let you continue integration with renovate on MegaLinter |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
@echoix When I open this discussion, 2025-08-05 GitHub PR seems unavailable !
I've found origin of issue (see screenshot chunk of PR 5916 report)
The PHP
realpath
return here false rather than expected full path of current working directory.I'll publish tomorrow a fix version of
bartlett/sarif-php-converters
1.3.1 that will avoid such failure.VCS provenance (new 1.3 feature) will not be available on SARIF report but version 1.3.x will act as previous version 1.2.0
Beta Was this translation helpful? Give feedback.
All reactions