Homec4science

Implement more consistent publishing rules for repositories

Authored by epriestley <git@epriestley.com> on Feb 19 2015, 19:38.

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

Committed
epriestley <git@epriestley.com>Feb 19 2015, 19:38
Pushed
aubortJan 31 2017, 17:16
Parents
rPH29fd3f136b88: Allow columns to be marked as nonmutable (so save() will not change them)
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH8599145b5eec: Implement more consistent publishing rules for repositories (authored by epriestley <git@epriestley.com>).Feb 19 2015, 19:38