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)
	Tue, Nov 19, 9:57 AM

	Unknown Object (File)
	Wed, Oct 23, 2:30 PM

	Unknown Object (File)
	Oct 21 2024, 8:46 PM

	Unknown Object (File)
	Oct 18 2024, 7:47 PM

	Unknown Object (File)
	Oct 15 2024, 5:18 AM

	Unknown Object (File)
	Oct 13 2024, 11:33 AM

	Unknown Object (File)
	Oct 12 2024, 11:48 PM

	Unknown Object (File)
	Oct 11 2024, 8:41 PM

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...