( )⚙ D8135 phabricator: refactor `phabread` to write all patches at once

This is an archive of the discontinued Mercurial Phabricator instance.

phabricator: refactor `phabread` to write all patches at once
ClosedPublic

Authored by mharbison72 on Feb 20 2020, 8:09 PM.

Details

Summary

This will be necessary to create a first class phabimport command. That
command requires a transaction, and will import all named patches within a
single transaction. But if Phabricator queries also happen within the
transaction, that leaves open the chance that an exception is raised, the
transaction is abandoned, and the next command that is run will complain about
needing to run hg recover.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.