Shipshape - various adjustments and fixes.

[barbun]

Changes

• Removed duplicated piece of code in scripts/validate/govcms-validate-illegal-files
• Flipped a check in scripts/validate/govcms-validate-permissions to explicitly assert for is_admin being set to false to avoid value 1 getting past the check.
• Adjusted regex pattern in shipshape.yml config for "[FILE] Disallowed permissions" check to target files more precisely.
• Adjusted regex pattern in shipshape.yml config for "[FILE] Detect module files in theme folder" check to target files more precisely.
• Adjusted path field values in shipshape.yml to remove "web" folder references as that can be set as a workdir when executing shipshape command. -- Removed "[FILE] Yaml lint theme files (no web prefix)" check as duplicate.
• Adjusted regex pattern in shipshape.yml config for "[FILE] Yaml lint theme files" as it was catching non-YAML files.

[barbun]

Hi @steveworley I reverted back the workdir related changes. We can brainstorm those later maybe. The rest of the changes should be OK