Paths

Table of Contentst

Fix tab complete in php 8
ClosedPublic
Actions

Authored by avivey on Jul 15 2023, 19:23.

Tags

None

Referenced Files

	F4051956: D25350.1746768422.diff
	Thu, May 8, 05:27

	F4049382: D25350.1746757358.diff
	Thu, May 8, 02:22

	F4049236: D25350.1746756770.diff
	Thu, May 8, 02:12

	F4032598: D25350.1746708214.diff
	Wed, May 7, 12:43

	F3991762: D25350.1746594335.diff
	Tue, May 6, 05:05

	F3984312: D25350.1746570359.diff
	Mon, May 5, 22:25

	F3979536: D25350.1746539212.diff
	Mon, May 5, 13:46

	F3946084: D25350.1746446255.diff
	Sun, May 4, 11:57

Subscribers

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15064: Make Phorge compatible with PHP 8.1/8.2/8.3/8.4

Commits

rARC8a2cb75d6308: Fix tab complete in php 8

Summary

Ref T15064

Test Plan

run arc shell-complete; arc shell-complete arc shell; Should suggest completions, not crash.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

avivey created this revision.Jul 15 2023, 19:23

Owners added a reviewer: O1: Blessed Committers.Jul 15 2023, 19:23

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan and 2 others. · View Herald TranscriptJul 15 2023, 19:23

avivey requested review of this revision.Jul 15 2023, 19:23

Harbormaster completed remote builds in B671: Diff 1124.Jul 15 2023, 19:23

mainframe98 subscribed.Jul 16 2023, 11:44

valerio.bozzolan added inline comments.Jul 18 2023, 07:36

src/toolset/workflow/ArcanistShellCompleteWorkflow.php
95	Finally we can remove the `(bool)`

remove bool

Harbormaster completed remote builds in B682: Diff 1144.Jul 18 2023, 15:18

avivey marked an inline comment as done.Jul 18 2023, 15:40

Thanks :) tested

lgtm

This revision is now accepted and ready to land.Jul 21 2023, 10:16

Closed by commit rARC8a2cb75d6308: Fix tab complete in php 8. · Explain WhyJul 21 2023, 16:46

This revision was automatically updated to reflect the committed changes.

avivey added a commit: rARC8a2cb75d6308: Fix tab complete in php 8.

Revision Contents
Changeset List

Path

Size

Packages

src/

toolset/

workflow/

ArcanistShellCompleteWorkflow.php

2 lines

O1: Blessed Committers

Diff 1150

View Options

src/toolset/workflow/ArcanistShellCompleteWorkflow.php

Fix tab complete in php 8ClosedPublicActions