Paths

Table of Contentst

Differential D2653

Modernize code to PHP 8.1 standard
ClosedPublic
Actions

Authored by dereckson on Apr 9 2022, 07:50.

Tags

None

Referenced Files

	F9558269: D2653.id6706.diff
	Sat, Jun 7, 11:45

	Unknown Object (File)
	Fri, Jun 6, 09:27

	Unknown Object (File)
	Wed, Jun 4, 01:44

	Unknown Object (File)
	Sun, Jun 1, 01:15

	Unknown Object (File)
	Sat, May 31, 13:22

	Unknown Object (File)
	Mon, May 26, 14:24

	Unknown Object (File)
	Wed, May 21, 19:15

	Unknown Object (File)
	Tue, May 20, 03:45

Subscribers

None

Details

Reviewers

Commits

rKCMDbaf6e15c21a8: Modernize code to PHP 8.1 standard
rKERUALD274a34db1539: Modernize code to PHP 8.1 standard

Summary

Don't use enum to avoid constructions like ExitCode::SUCCESS->value.

Test Plan

Run tests and phan

Diff Detail

Repository

rKERUALD Keruald libraries development repository

Lint

Lint Passed

Unit

Test Failures

Branch

modernize-commands-php81

Build Status

Buildable 4160
Build 4412: arc lint + arc unit

Unit TestsBroken
View All

	Time	Test
	0 ms	Keruald\Commands\Tests\CommandTest Keruald\Commands\Tests\CommandTest::testGetArgc Error: Class "Keruald\Commands\Display\ArrayDisplay" not found
	0 ms	Keruald\Commands\Tests\Display\ArrayDisplayTest Keruald\Commands\Tests\Display\ArrayDisplayTest::testOut Error: Class "Keruald\Commands\Display\ArrayDisplay" not found

Event Timeline

dereckson requested review of this revision.Apr 9 2022, 07:50

dereckson created this revision.

Harbormaster completed remote builds in B4160: Diff 6704.Apr 9 2022, 07:50

For the tests, the composer.json commit for all libraries hasn't been merged yet.

dereckson added a parent revision: D2547: Generate composer.json.Apr 9 2022, 07:53

Rebased

Harbormaster completed remote builds in B4161: Diff 6706.Apr 9 2022, 08:06

dereckson accepted this revision.Apr 9 2022, 08:07

This revision is now accepted and ready to land.Apr 9 2022, 08:07

Closed by commit rKERUALD274a34db1539: Modernize code to PHP 8.1 standard (authored by dereckson). · Explain WhyApr 9 2022, 08:07

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rKERUALD274a34db1539: Modernize code to PHP 8.1 standard.

dereckson added a commit: rKCMDbaf6e15c21a8: Modernize code to PHP 8.1 standard.Jul 2 2023, 18:53

Revision Contents
Changeset List

Path

Size

commands/

2 lines

2 lines

src/

34 lines

Display/

ArrayDisplay.php

8 lines

Diff 6704

commands/LICENSE

Loading...

commands/composer.json

Loading...

commands/src/Command.php

Loading...

commands/src/Display/ArrayDisplay.php

Loading...