- last update
about 1 year ago PHPLint Failed - last update
about 1 year ago PHPLint Failed - πΊπ¦Ukraine abramm Lutsk
I've faced the same issue and written a quick test for this.
Attaching both test only (which should fail) and test + prudloff's fix patches (which should pass tests and probably should be fine to merge).
- last update
about 1 year ago 7 pass - last update
about 1 year ago 7 pass - πΊπ¦Ukraine abramm Lutsk
Hmm that's interesting, the new CSSDeprecationAnalyzerTest test class haven't been picked up by CI.
Any ideas? Feels like I missed something totally simple. - last update
about 1 year ago 7 pass, 2 fail - last update
about 1 year ago 7 pass, 2 fail The last submitted patch, 8: 3324201-8-test-only.patch, failed testing. View results β
The last submitted patch, 8: 3324201-8-test-and-fix.patch, failed testing. View results β
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Status changed to Needs work
about 1 year ago 1:22pm 20 September 2023 - πΊπ¦Ukraine abramm Lutsk
You won this time, CI, but it's not over!
The test have failed due to the way I've usedtempnam
for creating a temporary directory. - last update
about 1 year ago 7 pass, 1 fail - last update
about 1 year ago 8 pass - last update
about 1 year ago 8 pass - Status changed to Needs review
about 1 year ago 1:37pm 20 September 2023 - πΊπ¦Ukraine abramm Lutsk
Thank you CI, you were a worthy opponent, our battle have been legendary.
Ready for review.
- Status changed to Fixed
about 1 year ago 5:16pm 20 September 2023 - ππΊHungary GΓ‘bor Hojtsy Hungary
Looks good to me, thanks for the elaborate test coverage!
Automatically closed - issue fixed for 2 weeks with no activity.