Closed Priyansh61 closed 1 year ago
@arminpatel Dont merge it because the scripts are being executed in global env instead of venv if you think of any way do let me know
@arminpatel @Tusharjain123 Any suggestions?
@Priyansh61 imo using python to use a subprocess to source a file would be difficult and really roundabout way to do it. How about using a simple shell script. Or rather doing this at some later stage?
@Priyansh61 I will prefer using a shell script because using a subprocess in Python to source a file will be a bit complex as compared to a shell script. With shell script, we can directly execute the source command to load the environment variables.
@arminpatel @Tusharjain123 Your suggestion really helped! Take a look now and just check if the venv gets activated correctly now because we dont want to mess things up, since there will be a lot of dependecies in the near future else they will be installed globally
Overview
Essential Checklist
Proof that changes are correct
PR Pointers