why would not return the resulting commit id?   more natural that way.