should have commited sooner