HomePhorge

Implement more consistent publishing rules for repositories

Description

Implement more consistent publishing rules for repositories

Summary:
Ref T7298. We are currently inconsistent about when we publish feed, email, notifications, audits and Herald rules.

Specifically, there are two settings which impact these things:

  • The "importing" flag, which is set when we're importing old commits.
  • The "herald-disabled" flag, which was expanded in scope some time ago and now actually means "disable publishing".

Various parts of the pipeline were checking only one of these flags. Instead, all of them should check both.

(For example, we should never email users about importing repositories, nor trigger audits on them.)

Test Plan: See next revision.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T7298

Differential Revision: https://secure.phabricator.com/D11826

Details

Provenance
epriestleyAuthored on Feb 19 2015, 10:38 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP29fd3f136b88: Allow columns to be marked as nonmutable (so save() will not change them)
Branches
Loading...
Tags
Loading...

Event Timeline