Closed shmolf closed 6 months ago
The main purpose of this PR, is to normalize the casing (upper/lower case) of the different categories, so that regardless of what convention the external repository uses, this aggregation will be consistent.
Addressing https://github.com/Lissy93/portainer-templates/issues/17
While QA'ing the changes locally, I saw the validate.py would fail because the Bookstack template was missing the image property.
validate.py
Bookstack
image
Validation error: 'image' is a required property Title of invalid template: Bookstack
This was not obvious as to who the offending template was, so I made a minor change to more easily identify which template caused the issue.
@Lissy93 Is this PR acceptable?
If not, I'm fine just closing it.
The main purpose of this PR, is to normalize the casing (upper/lower case) of the different categories, so that regardless of what convention the external repository uses, this aggregation will be consistent.
Addressing https://github.com/Lissy93/portainer-templates/issues/17
While QA'ing the changes locally, I saw the
validate.py
would fail because theBookstack
template was missing theimage
property.This was not obvious as to who the offending template was, so I made a minor change to more easily identify which template caused the issue.