added the submodule the right way?