Page MenuHomePhabricator

Address some PHP8 incompatibilities - Arcanist
ClosedPublic

Authored by cspeckmim on May 29 2023, 11:40 PM.
Tags
None
Referenced Files
F14076920: D21870.diff
Thu, Nov 21, 8:21 PM
Unknown Object (File)
Fri, Nov 15, 1:43 PM
Unknown Object (File)
Tue, Nov 12, 7:30 AM
Unknown Object (File)
Fri, Nov 8, 3:09 PM
Unknown Object (File)
Oct 19 2024, 4:59 AM
Unknown Object (File)
Oct 16 2024, 7:58 AM
Unknown Object (File)
Oct 12 2024, 11:11 AM
Unknown Object (File)
Oct 9 2024, 2:49 PM
Subscribers

Details

Summary

Found a few more issues while playing around with arc.

Refs T13588

Test Plan
  • Run arc look and variants with subcommands
  • Run arc diff while not on a git branch

Diff Detail

Repository
rARC Arcanist
Branch
cspeck-php8-arc
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 25831
Build 35659: arc lint + arc unit

Unit TestsFailed

TimeTest
2,834 msArcanistCoffeeLintLinterTestCase::testLinter
Lint emitted an unexpected set of messages for file "no_trailing_whitespace.lint-test". MISSING MESSAGES Message with severity "error" at "1:0"
189 msArcanistCppcheckLinterTestCase::testLinter
Lint emitted an unexpected set of messages for file "zblair.lint-test". MISSING MESSAGES Message with severity "error" at "10:<null>"
1,575 msArcanistJSHintLinterTestCase::testLinter
Lint emitted an unexpected set of messages for file "too-many-errors.lint-test". MISSING MESSAGES Message with severity "disabled" at "2:22" (E043)
917 msArcanistJscsLinterTestCase::testLinter
Lint emitted an unexpected set of messages for file "curly-brace.lint-test". MISSING MESSAGES Message with severity "error" at "2:3"
1,385 msArcanistLesscLinterTestCase::testLinter
Lint emitted an unexpected set of messages for file "javascript.lint-test". MISSING MESSAGES (No messages.)
View Full Test Results (8 Failed · 41 Passed · 8 Skipped)

Event Timeline

cspeckmim held this revision as a draft.

Update version check for Cppcheck to include optional bugfix version

Looking at the rest of the UT failures I think those are due to updated/newer versions of linters emitting slightly different results.

This revision is now accepted and ready to land.May 30 2023, 4:29 PM
This revision was landed with ongoing or failed builds.May 30 2023, 4:39 PM
This revision was automatically updated to reflect the committed changes.