Add a hook method that is called after `ArcanistLinter::setEngine`
Needs ReviewPublic
Actions

Authored by joshuaspence on May 16 2019, 1:57 AM.

Tags

None

Referenced Files

	F13100647: D20523.id.diff
	Fri, Apr 26, 7:16 PM

	Unknown Object (File)
	Wed, Apr 24, 11:06 PM

	Unknown Object (File)
	Wed, Apr 24, 3:26 PM

	Unknown Object (File)
	Fri, Apr 19, 2:37 PM

	Unknown Object (File)
	Fri, Apr 19, 3:13 AM

	Unknown Object (File)
	Fri, Apr 19, 3:13 AM

	Unknown Object (File)
	Fri, Apr 19, 3:13 AM

	Unknown Object (File)
	Thu, Apr 11, 4:21 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Summary

Similar to D19730 and D20514, I need a hook that can be used in ArcanistDockerContainerLinterProxy, a third-party linter wrapper. Instead of removing final from ArcanistLinter::setEngine, I figured it was better to create an overridable didSetEngine method, since if subclasses forget to call parent::setEngine() from their own implementation of the setEngine method then bad things might happen?

Test Plan

N/A