This should be safe to remove now. Also removes DagitRemoteRunLauncher.
- R1:66813263dc30: [core] remove execute_run_in_process
My only question is that are we going to break anyone relying on the remote run launcher. Do need to go through a deprecation cycle?
Good question, I'm not sure if anyone is using it anymore. I think other changes have made the places this was used as a workaround go away.
can maybe keep it around by changing this to launch run