submodule was a bad idea I think