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

	F13216152: D11330.id27223.diff
	Fri, May 17, 10:38 PM

	F13195208: D11330.diff
	Sun, May 12, 10:13 PM

	F13179393: D11330.diff
	Wed, May 8, 9:10 PM

	Unknown Object (File)
	Sat, May 4, 6:43 PM

	Unknown Object (File)
	Wed, May 1, 12:03 AM

	Unknown Object (File)
	Apr 17 2024, 2:49 PM

	Unknown Object (File)
	Apr 16 2024, 2:48 AM

	Unknown Object (File)
	Apr 5 2024, 8:47 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Commits

rARC6694834f29d6: Minor tidying of `ArcanistPhutilLibraryLinter`

Summary

Self-explanatory.

Test Plan

Eyeball it.

Diff Detail

Repository

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 3763
Build 3774: [Placeholder Plan] Wait for 30 Seconds

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–215	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 27223

src/lint/linter/ArcanistPhutilLibraryLinter.php

Loading...