Page MenuHomePhabricator
Differential D21870

Address some PHP8 incompatibilities - Arcanist
ClosedPublic
Actions

Authored by cspeckmim on May 29 2023, 11:40 PM.
Tags
None
Referenced Files
F18876895: D21870.diff
Thu, Nov 6, 4:48 AM
F18850303: D21870.diff
Thu, Oct 30, 3:41 PM
F18814945: D21870.diff
Tue, Oct 21, 12:12 AM
F18810250: D21870.diff
Sun, Oct 19, 8:34 PM
F18708551: D21870.id.diff
Sep 28 2025, 10:08 PM
F18679109: D21870.id52172.diff
Sep 26 2025, 12:31 AM
F18631587: D21870.diff
Sep 16 2025, 3:48 PM
F18629071: D21870.id52166.diff
Sep 16 2025, 7:56 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T13588: PHP 8 Compatibility
Commits
rARCe50d1bc4eaba: Address some PHP8 incompatibilities - Arcanist
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cspeckmim created this revision.May 29 2023, 11:40 PM
cspeckmim held this revision as a draft.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 29 2023, 11:40 PM
Herald added a subscriber: epriestley. · View Herald Transcript
Harbormaster completed remote builds in B25825: Diff 52166.May 29 2023, 11:40 PM
cspeckmim updated this revision to Diff 52172.May 30 2023, 3:22 PM

Update version check for Cppcheck to include optional bugfix version

Harbormaster completed remote builds in B25831: Diff 52172.May 30 2023, 3:22 PM
cspeckmim edited the summary of this revision. (Show Details)May 30 2023, 3:34 PM
cspeckmim added a task: T13588: PHP 8 Compatibility.
cspeckmim published this revision for review.May 30 2023, 4:26 PM

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

epriestley accepted this revision.May 30 2023, 4:29 PM
This revision is now accepted and ready to land.May 30 2023, 4:29 PM
Herald added a subscriber: Korvin. · View Herald TranscriptMay 30 2023, 4:29 PM
This revision was landed with ongoing or failed builds.May 30 2023, 4:39 PM
Closed by commit rARCe50d1bc4eaba: Address some PHP8 incompatibilities - Arcanist (authored by cspeckmim). · Explain Why
This revision was automatically updated to reflect the committed changes.
cspeckmim added a commit: rARCe50d1bc4eaba: Address some PHP8 incompatibilities - Arcanist.

Revision Contents
Changeset List

PathSize
src/
browse/
query/
2 lines
lint/
linter/
2 lines
10 lines
ref/
10 lines
repository/
api/
2 lines
workflow/
4 lines

Diff 52177

View Options

src/browse/query/ArcanistBrowseObjectNameURIHardpointQuery.php

Loading...
View Options

src/lint/linter/ArcanistCppcheckLinter.php

Loading...
View Options

src/lint/linter/ArcanistScriptAndRegexLinter.php

Loading...
View Options

src/ref/ArcanistRefView.php

Loading...
View Options

src/repository/api/ArcanistGitAPI.php

Loading...
View Options

src/workflow/ArcanistWorkflow.php

Loading...
Phabricator · Built by Phacility