Paths

Table of Contentst

Differential D11330

Minor tidying of `ArcanistPhutilLibraryLinter`
ClosedPublic
Actions

Authored by joshuaspence on Jan 11 2015, 8:16 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 20, 6:20 PM

	Unknown Object (File)
	Thu, Dec 19, 12:45 AM

	Unknown Object (File)
	Mon, Dec 16, 5:30 PM

	Unknown Object (File)
	Sun, Dec 1, 3:45 AM

	Unknown Object (File)
	Wed, Nov 27, 3:24 PM

	Unknown Object (File)
	Tue, Nov 26, 6:16 AM

	Unknown Object (File)
	Tue, Nov 26, 2:53 AM

	Unknown Object (File)
	Tue, Nov 26, 2:51 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Commits

rARC6694834f29d6: Minor tidying of `ArcanistPhutilLibraryLinter`

Summary

Self-explanatory.

Test Plan

Eyeball it.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 27223.Jan 11 2015, 8:16 PM

joshuaspence retitled this revision from to Minor tidying of `ArcanistPhutilLibraryLinter`.

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 11 2015, 8:16 PM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley accepted this revision.Jan 11 2015, 8:23 PM

epriestley edited edge metadata.

epriestley added inline comments.

src/lint/linter/ArcanistPhutilLibraryLinter.php
193–216	Consider `pht()`'ing the entire block in cases like this. In other languages, it may take more/fewer/different sentences to express an idea, which are easier to translate in a single large block.

This revision is now accepted and ready to land.Jan 11 2015, 8:23 PM

Closed by commit rARC6694834f29d6: Minor tidying of `ArcanistPhutilLibraryLinter` (authored by joshuaspence, committed by joshuaspence). · Explain WhyJan 15 2015, 11:24 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

lint/

linter/

ArcanistPhutilLibraryLinter.php

112 lines

Diff 27391

src/lint/linter/ArcanistPhutilLibraryLinter.php

Loading...