Paths

Table of Contentst

Never try to run README as a commit hook
ClosedPublic
Actions

Authored by epriestley on May 5 2014, 3:43 PM.

Tags

None

Referenced Files

	F19561197: D8981.id21313.diff
	Sat, Jan 31, 9:51 AM

	F19543524: D8981.id21313.diff
	Thu, Jan 22, 8:28 PM

	F19506486: D8981.id21306.diff
	Jan 9 2026, 4:47 PM

	F18943694: D8981.id21306.diff
	Nov 11 2025, 2:30 PM

	F18943693: D8981.id21313.diff
	Nov 11 2025, 2:30 PM

	F18849657: D8981.id.diff
	Oct 30 2025, 11:13 AM

	F18839963: D8981.id21313.diff
	Oct 27 2025, 8:14 PM

	F18803397: D8981.id21306.diff
	Oct 17 2025, 11:23 PM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan
chad

Maniphest Tasks

T4960: Never try to run README as a commit hook

Commits

Restricted Diffusion Commit
rP74faacee4d4a: Never try to run README as a commit hook

Summary

Fixes T4960. Users chmod +x this, and then bash chokes on it.

Phabricator "owns" this file anyway, so there is no real ambiguity here: this should never be a hook script.

Test Plan

Did chmod +x README.
Made a commit.
Added z.sh, got blocked.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 21306.May 5 2014, 3:43 PM

epriestley retitled this revision from to Never try to run README as a commit hook.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T4960: Never try to run README as a commit hook.

Herald added a subscriber: epriestley. · View Herald TranscriptMay 5 2014, 3:43 PM

Harbormaster completed remote builds in B248: Diff 21306.May 5 2014, 3:44 PM

chad accepted this revision.May 5 2014, 4:30 PM

chad added a reviewer: chad.

This revision is now accepted and ready to land.May 5 2014, 4:30 PM

Closed by commit rP74faacee4d4a (authored by @epriestley).

Harbormaster edited this Differential Revision.May 5 2014, 8:53 PM

Harbormaster completed remote builds in B248: Diff 21306.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

engine/

DiffusionCommitHookEngine.php

14 lines

Diff 21313

View Options

src/applications/diffusion/engine/DiffusionCommitHookEngine.php

Never try to run README as a commit hookClosedPublicActions