Differential D2164

Document File::from return value as static
ClosedPublic
Actions

Authored by dereckson on Jan 29 2020, 03:10.

Tags

None

Referenced Files

	F3914946: D2164.diff
	Thu, Dec 19, 17:55

	Unknown Object (File)
	Tue, Dec 17, 12:06

	Unknown Object (File)
	Tue, Dec 17, 06:05

	Unknown Object (File)
	Mon, Dec 16, 16:34

	Unknown Object (File)
	Wed, Dec 11, 10:21

	Unknown Object (File)
	Sat, Dec 7, 07:31

	Unknown Object (File)
	Thu, Dec 5, 15:32

	Unknown Object (File)
	Thu, Dec 5, 04:11

Subscribers

None

Details

Reviewers

Commits

rKOT42911b95f98d: Document File::from return value as static

Summary

This change allows Phan to correctly analyze the methods availability.

This is a follow-up for 7266cfd77f31.

Test Plan

Without the change, Phan outputs:

$ phan
src/Registration/PSR4/Autoloader.php:51 PhanUndeclaredMethod Call to undeclared method \Keruald\OmniTools\IO\File::tryInclude
tests/Reflection/CodeFileTest.php:41 PhanUndeclaredMethod Call to undeclared method \Keruald\OmniTools\IO\File::canBeIncluded

After the change, it doesn't prints these messages.

Diff Detail

Repository

rKOT Keruald OmniTools

Lint

Lint Passed

Unit

Test Failures

Branch

master

Build Status

Buildable 3365
Build 3614: arc lint + arc unit

Unit TestsBroken
View All

	Time	Test
	0 ms	/usr/home/dereckson/dev/keruald/omnitools/tests/IO/FileTest.php

Event Timeline

dereckson requested review of this revision.Jan 29 2020, 03:10

dereckson created this revision.

Harbormaster completed remote builds in B3365: Diff 5453.Jan 29 2020, 03:10

dereckson accepted this revision.Jan 29 2020, 03:11

This revision is now accepted and ready to land.Jan 29 2020, 03:11

This revision was landed with ongoing or failed builds.Jan 29 2020, 03:11

Closed by commit rKOT42911b95f98d: Document File::from return value as static (authored by dereckson). · Explain Why

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rKOT42911b95f98d: Document File::from return value as static.

Revision Contents
Changeset List

Path

Size

src/

IO/

3 lines

Diff 5453

src/IO/File.php

Loading...