This branch attempts to address all existing TODOs within the PGE codebase and either address them, leave them (as true tasks to be done later), or remove them if they no longer seemed relevant. I have made an attempt to justify each removal in the corresponding commit.
As part of the resolution for on TODO, redundant validation checks on the ISO xml template that were duplicated in each PGE module have been refactored into base_pge.py
Resolution of another TODO has resulted in a change to all Dockerfiles such that we now always recursively open up permissions of each directory within the default working area of the container.
Affected Issues
Resolves #515
Testing
Unit tests have been updated to account for changes to ISO xml template validation
Ran integration testing of each PGE type on this branch to ensure changes to file system permissions did not break anything
Description
Affected Issues
Testing