should have commited earlier