Code review comment for lp:~jelmer/brz/location-purpose

This branch has the hook change in as well. Not quite sure I understand the reasoning here, but seems basically reasonable? Tests use 'push' as a purpose, but the documented values are 'read' 'write' None?