Differential D2536

Simplify maxPropertyLength computation
ClosedPublic
Actions

Authored by DorianWinty on Feb 17 2022, 21:18.

Tags

None

Referenced Files

	F22603022: D2536.diff
	Thu, Jan 29, 20:59

	F22510593: D2536.id.diff
	Thu, Jan 29, 02:41

	Unknown Object (File)
	Sat, Jan 24, 21:31

	Unknown Object (File)
	Sat, Jan 24, 00:40

	Unknown Object (File)
	Wed, Jan 21, 20:19

	Unknown Object (File)
	Wed, Jan 21, 16:15

	Unknown Object (File)
	Wed, Jan 21, 05:03

	Unknown Object (File)
	Tue, Jan 20, 09:47

Subscribers

None

Details

Reviewers

Commits

rKREPORT3c06786204d0: Simplify maxPropertyLength computation

Summary

maxPropertyLength modify for making more easy

Diff Detail

Repository

rKREPORT Keruald Report

Lint

Lint Passed

Unit

Tests Passed

Branch

mark_update

Build Status

Buildable 3963
Build 4215: arc lint + arc unit

Event Timeline

DorianWinty requested review of this revision.Feb 17 2022, 21:18

DorianWinty created this revision.

Harbormaster completed remote builds in B3962: Diff 6399.Feb 17 2022, 21:18

a

Harbormaster completed remote builds in B3963: Diff 6400.Feb 17 2022, 21:18

DorianWinty added a reviewer: dereckson.Feb 17 2022, 21:19

dereckson added inline comments.Feb 17 2022, 21:22

src/Output/MarkdownOutput.php
34	min() instead of max() if the goal is to get at least 8 characters?

dereckson retitled this revision from modify maxpropertylength to Simplify maxPropertyLength computation.Feb 17 2022, 21:23

DorianWinty added inline comments.Feb 17 2022, 21:28

src/Output/MarkdownOutput.php
34	if i put min() when $propertyMaxLength is more than 8 the number will stay at 8 but if i use max if $propertyMaxLength > 8 then it will be the value of the property

dereckson accepted this revision.Feb 17 2022, 21:45

This revision is now accepted and ready to land.Feb 17 2022, 21:45

Indeed, mea culpa.

Closed by commit rKREPORT3c06786204d0: Simplify maxPropertyLength computation (authored by DorianWinty, committed by dereckson). · Explain WhyFeb 17 2022, 21:47

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rKREPORT3c06786204d0: Simplify maxPropertyLength computation.

Revision Contents
Changeset List

Path

Size

src/

Output/

MarkdownOutput.php

4 lines

Diff 6400

src/Output/MarkdownOutput.php

Loading...