Differential D19298

Remove "Large Changes" documentation and make some minor behavioral improvements
ClosedPublic
Actions

Authored by epriestley on Apr 5 2018, 1:29 PM.

Tags

None

Referenced Files

	F13261220: D19298.diff
	Mon, May 27, 12:48 AM

	F13247924: D19298.diff
	Thu, May 23, 11:31 PM

	F13245530: D19298.diff
	Thu, May 23, 6:05 AM

	F13238061: D19298.id46182.diff
	Tue, May 21, 4:53 PM

	F13227223: D19298.diff
	Mon, May 20, 1:49 AM

	F13222188: D19298.diff
	Sun, May 19, 3:25 AM

	F13204731: D19298.diff
	Wed, May 15, 12:53 AM

	F13187841: D19298.diff
	Sat, May 11, 4:46 AM

View All 90 Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13110: Plans: Hefty Differential revisions, draft state transitions, and exotic interactions
Commits: rPf01c2e36948b: Remove "Large Changes" documentation and make some minor behavioral improvements

Summary

Depends on D19296. Ref T13110.

Remove the "Large Changesets" documentation since we now degrade very large changesets and I don't have any evidence that anyone has ever tried to follow any of the recommendations in this document.
Remove references to it.
When an older revision doesn't have denormalized size information on the Revision object itself, don't render a scale element (instead of rendering a bogus one).
Try to improve terminology consistency around "Large Change" (100-1000 files) vs "Very Large Change" (1000+ files) vs "Enormous Change" (too large to hold in memory).

Test Plan

Viewed revisions; grepped for documentation.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 5 2018, 1:29 PM

Harbormaster completed remote builds in B20004: Diff 46182.Apr 5 2018, 1:30 PM

epriestley requested review of this revision.Apr 5 2018, 1:30 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 5 2018, 1:40 PM

Closed by commit rPf01c2e36948b: Remove "Large Changes" documentation and make some minor behavioral improvements (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

conduit/

DifferentialCreateRawDiffConduitAPIMethod.php

5 lines

controller/

DifferentialRevisionViewController.php

2 lines

storage/

DifferentialRevision.php

6 lines

view/

DifferentialRevisionListView.php

9 lines

docs/

flavor/

writing_reviewable_code.diviner

3 lines

user/

userguide/

differential.diviner

2 lines

differential_large_changes.diviner

Diff 46186

src/applications/differential/conduit/DifferentialCreateRawDiffConduitAPIMethod.php

Loading...

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...

src/applications/differential/storage/DifferentialRevision.php

Loading...

src/applications/differential/view/DifferentialRevisionListView.php

Loading...

src/docs/flavor/writing_reviewable_code.diviner

Loading...

src/docs/user/userguide/differential.diviner

Loading...

src/docs/user/userguide/differential_large_changes.diviner

Loading...