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
F15717853: D21870.diff
Thu, May 29, 6:14 AM
F15701479: D21870.diff
Wed, May 28, 7:10 PM
Unknown Object (File)
Fri, May 9, 11:45 PM
Unknown Object (File)
Apr 28 2025, 10:55 AM
Unknown Object (File)
Apr 24 2025, 5:23 AM
Unknown Object (File)
Apr 23 2025, 6:54 PM
Unknown Object (File)
Apr 21 2025, 11:27 AM
Unknown Object (File)
Apr 11 2025, 12:34 PM
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