Page MenuHomePhorge

Herald Transcripts: Use μs instead of us for microseconds
AcceptedPublic

Authored by aklapper on Mon, Mar 31, 21:17.

Details

Reviewers
valerio.bozzolan
Group Reviewers
O1: Blessed Committers
Summary

Use Unicode abbreviation μs for microseconds instead of ASCII us which made me wonder for a while what that is.

mb_chr is available since PHP 7.2.0 (which Phorge requires). Entering μ directly makes lint bark about "Bad Charset".

Test Plan

Go to http://phorge.localhost/herald/transcript/1/profile/ and look at values in the "Cost" column.
Works with PHP 8.3; no clue which prehistoric PHP versions may have problems.

Diff Detail

Repository
rP Phorge
Branch
heraldMicroseconds (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1835
Build 1835: arc lint + arc unit