Page MenuHomePhabricator

Mark IMPORTED_CHANGE more consistently
ClosedPublic

Authored by epriestley on Nov 1 2013, 3:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 28, 1:46 PM
Unknown Object (File)
Fri, Dec 20, 9:45 PM
Unknown Object (File)
Tue, Dec 17, 8:29 PM
Unknown Object (File)
Mon, Dec 9, 5:53 AM
Unknown Object (File)
Sat, Dec 7, 8:24 PM
Unknown Object (File)
Fri, Dec 6, 10:31 PM
Unknown Object (File)
Thu, Dec 5, 7:29 AM
Unknown Object (File)
Wed, Dec 4, 12:56 PM
Subscribers

Details

Reviewers
btrahan
Commits
Restricted Diffusion Commit
rPd1649d176f67: Mark IMPORTED_CHANGE more consistently
Summary

See https://github.com/facebook/phabricator/issues/425. There are some ways that the change parsers may not reach finishParse(), but we now need them to in order to mark the commit imported, advance the progress bar, and eventually kick the repository out of IMPORTING status.

Take all the copy/pasted code in the parsers and move it into the parent. Specifically, this is:

  • Printing a status message about starting a parse;
  • checking for bad commits;
  • queueing the next parse stage; and
  • marking the import step complete.
Test Plan

Used reparse.php --change to reparse Git, SVN and Mercurial repos.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Nov 1 2013, 3:57 PM

Smaller diff.