is it correct? and it is actually needed? we have git log