Remove old hardcoded list of repo excludes

[Flipez]

In the voxpupuli.yml we keep a (short) list of repos we exlcude. Since we now support configuration we should remove that once every repo has implemented the enabled: false flag to the .sync.yml.

List:

• ⌛puppet-bacula
• ⌛puppet-nagios_providers
• ⌛puppet-iis
• ⌛puppet-syntax
• ⌛puppet-blacksmith
• ⌛puppet-mode

[ekohl]

I'd like to voice my concern about this. My goal is to get rid of .sync.yml as much as possible. Ideally, all modules follow the same standards.

Have you considered using a repository label in Github? Something like vp-tasks-skip?

[bastelfreak]

I thought about the repository labels/topics as well. With the .sync.yml we have the advantage of a review/permission management. I just checked https://docs.github.com/en/github/administering-a-repository/managing-repository-settings/classifying-your-repository-with-topics and only admins can currently manage topis, so that might not be too bad.

[Flipez]

The advantage of a YAML file (whatever it is named) is in my opinion the possibility for different options (eg. disable a specific check, disable comments for a specific check).

Currently we only support three flags but I think this will become more and more as we also might introduce new checks or more comments/actions for different things. Managing them all through topics would be pretty confusing I think as there is no possibility for nesting them.

I not necessarily need the configuration to happen in the .sync.yml but I would like to keep in in the form of a file (maybe a vpt.yml or similiar?

[ekohl]

Perhaps #304 (comment) was meant for this issue.