Homec4science

Provide an alternate, more general "closeable" flag for commits

Authored by epriestley <git@epriestley.com> on Jan 16 2014, 23:28.

Description

Provide an alternate, more general "closeable" flag for commits

Summary:
Ref T4327. This provides a more general RefCursor-based way to identify closeable commits, and moves away from the messy seenOnBranches stuff. Basically:

  • When a closeable ref (like the branch "master") is updated, query the VCS for all commits which are ancestors of the new ref, but not ancestors of the existing closeable heads we previously knew about. This is basically all the commits which have been merged or moved onto the closeable ref.
  • Take these commits and set the "closeable" flag on the ones which don't have it yet, then queue new tasks to reprocess them.

I haven't removed the old stuff yet, but will do that shortly.

Test Plan:

  • Ran bin/repository discover and bin/repository refs on a bunch of different VCSes and VCS states. The effects seemed correct (e.g., new commits were marked as closeable).

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T4327

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

Details

Committed
epriestley <git@epriestley.com>Jan 17 2014, 20:48
Pushed
aubortJan 31 2017, 17:16
Parents
rPHf4b9efe2562b: Introduce ref cursors for repository parsing
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHcec3f44b900e: Provide an alternate, more general "closeable" flag for commits (authored by epriestley <git@epriestley.com>).Jan 17 2014, 20:48