Differential D19071

In Drydock log views, respect newlines
ClosedPublic
Actions

Authored by epriestley on Feb 13 2018, 11:36 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Apr 19, 2:49 AM

	Unknown Object (File)
	Fri, Apr 19, 2:49 AM

	Unknown Object (File)
	Fri, Apr 19, 2:49 AM

	Unknown Object (File)
	Wed, Apr 17, 2:16 AM

	Unknown Object (File)
	Thu, Apr 11, 4:06 AM

	Unknown Object (File)
	Sat, Mar 30, 6:48 PM

	Unknown Object (File)
	Sat, Mar 30, 6:48 PM

	Unknown Object (File)
	Sat, Mar 30, 6:48 PM

View All Files

Subscribers

Restricted Owners Package

Tokens

"Love" token, awarded by jcox.

Details

Reviewers: None
Maniphest Tasks: T13073: Plans: Drydock for normal software use cases where builds take more than 45 seconds
Commits: rP3ec80a36dbee: In Drydock log views, respect newlines

Summary

Depends on D19070. Ref T13073. Some messages contain an interesting story or a clever anecdote. Respect newlines during rendering to preserve authorial intent.

Test Plan

Viewed a message with linebreaks and could still read it.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 13 2018, 11:36 AM

Owners added a subscriber: Restricted Owners Package.Feb 13 2018, 11:36 AM

Harbormaster completed remote builds in B19518: Diff 45715.Feb 13 2018, 11:37 AM

epriestley requested review of this revision.Feb 13 2018, 11:37 AM

epriestley added a child revision: D19072: While waiting for a "bin/drydock" lease to activate, entertain the user with log output.Feb 13 2018, 11:50 AM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 13 2018, 9:12 PM

Closed by commit rP3ec80a36dbee: In Drydock log views, respect newlines (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

jcox awarded a token.Feb 13 2018, 9:14 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

drydock/

view/

DrydockLogListView.php

1 line

Diff 45727

View Options

src/applications/drydock/view/DrydockLogListView.php

In Drydock log views, respect newlinesClosedPublicActions