Hi Ali, I am accepting this PR with the changes you have made so far. Please go ahead and merge the branch, then delete the branch. Thanks !
Hi Ali, here is my first round of feedback. Please take a look at these when you can then I will do a final review to make sure there is nothing that I missed.
It's bad practice to import * so I would remove this line. Generally you only want to import what you need and nothing more.
this is not needed as the following line will allow for all files inside of C:\Program Files\QGIS 3.34.1\apps\Python39 to be accessible in Command Prompt or Powershell
Is there a reason you make a copy instead of just renaming the executable? If this is added to the path variable, your command prompt might point to this version of python 3.9 instead of the user/system installation of python 3.9 if it exists.
rename from handle_mtl_ds_workflow.py to montreal_buildings_workflow.py, quebec_buildings_workflow.py, or building_cleanup_workflow.py depending on what you feel fits the best. Call the section Montreal Building Workflow corresponding to the name of the file
Rename scrub_layer_class.py to scrub_layer.py and call this section ScrubLayer following the class name
I would remove the 'non-object-oriented project' and just say that it is a continuation of the project and provide the link
5e01f4eb7f
Merge pull request 'Remove build from setup.py and add it to requirements.txt' (#69) from fix/fix-build into main