Validate features

[LDannijs]

Summary

I stumbled upon the issues from @johanstokking and @KrishnaIyer (#108 and #405) and decided to give it a crack. I am very curious to hear back if these implementations are correct, they would help out a lot.

Changes

• Johan's issue:

  • Check if there are vendors in the vendor folder which are not in vendor/index.yaml.
  • Ability to pass an argument to the validation script to only validate a single vendor
    • Updated README with explanation

• Krishna's issue:

  • Check if the vendorProfileID is unique per vendor (Now this checks for 2 things: is the value unique and is it missing? if it's not unique it will stop the code, and if its missing it will just log it and continue)
  • Also updated every vendorProfileID to match this new feature to the validation

• Own changes:

  • Updated the PR template slightly
  • Improved the vendorProfileID explanation a bit further to make it even clearer.

Notes for Reviewers

@johanstokking I had a discussion with @Jaime-Trinidad about how much we should check the vendorProfileID and decided to just check for uniqueness and presence. Checking it against the vendorID was a bit outside of the scope since not every vendor adds it.

You asked about EIRP checking as well but i heard there can be some special exceptions so i left that out for now too.

[LDannijs]

I've added another small feature: add a pre-commit message that runs the validation on it's own and displays the error if the validation failed:

I sometimes see users failing to run the validation themselves so this might help to negate that instead of Jaime having to do it himself. I'm not entirely sure if this is something we'd need but I'll be happy to hear feedback.

[LDannijs]

I'm aware not everything is properly implemented, had to fight my script a bit (which will be removed), the schema is missing hardwareVersion, some files are missing proper formatting, yadayada. Will continue this later.

[LDannijs]

@johanstokking i am pretty sure the implementation will have some problems but just as check if this is the correct direction.

[LDannijs]

Alright @johanstokking think I finally understood it now (apologies) I now removed the profileIDs from vendors where vendorProfileID was a repeating id, a vendorid, a made up number, etc.

I think I messed up cause, yes, I initially created a check for if the vendorProfileID was unique, but I rolled back that change.