Add "random_int" and "random_bytes" as builtins so the linter stops complaining about them
ClosedPublic
Actions

Authored by epriestley on Feb 20 2019, 10:59 PM.

Tags

None

Referenced Files

	F18758801: D20202.id48236.diff
	Sun, Oct 5, 11:46 PM

	F18696398: D20202.id48237.diff
	Sat, Sep 27, 7:34 AM

	F18696021: D20202.diff
	Sat, Sep 27, 6:54 AM

	F18044334: D20202.diff
	Aug 3 2025, 7:05 AM

	F17865000: D20202.diff
	Jul 28 2025, 2:48 AM

	F17630851: D20202.diff
	Jul 10 2025, 12:01 PM

	Unknown Object (File)
	May 7 2025, 4:31 AM

	Unknown Object (File)
	May 5 2025, 11:51 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Commits

rPHU671ec8fb8f7a: Add "random_int" and "random_bytes" as builtins so the linter stops complaining…

Summary

There's a "right" way to do this by regenerating this file from PHPCompatInfo, but that requires installing composer and a whole lot of legwork. For now, I think we only actually care about these two functions, so I just added them manually.

Test Plan

On PHP7, ran arc lint on Filesystem.php and got no warnings. This change should also stop us from getting warnings on earlier versions of PHP.

Diff Detail

Repository

rPHU libphutil

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 20 2019, 10:59 PM

Harbormaster completed remote builds in B22093: Diff 48236.Feb 20 2019, 10:59 PM

epriestley requested review of this revision.Feb 20 2019, 10:59 PM

amckinley accepted this revision.Feb 20 2019, 11:00 PM

This revision is now accepted and ready to land.Feb 20 2019, 11:00 PM

Closed by commit rPHU671ec8fb8f7a: Add "random_int" and "random_bytes" as builtins so the linter stops complaining… (authored by epriestley). · Explain WhyFeb 20 2019, 11:03 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

php_compat_info.json

Diff 48237

View Options

resources/php_compat_info.json

Add "random_int" and "random_bytes" as builtins so the linter stops complaining about themClosedPublicActions