Pull requests actions

[Ash258]

Triggers

• [x] Opened PR
• [x] Opened PR from fork
• [x] Comment❓❓
  • Anyone could comment to trigger this action
  • When PR is updated, then anyone could comment something like: /verify to trigger action again
• [ ] Cleanup Test-PRFile function
  • [ ] Export checks into own funciton with $status, $manfiestOb, $manifestGci

Checks

1. [x] Description
2. [ ] License (Add note to children list, about what is wrong)
   • License property exists, but it is not valid SPDX identifier
3. [x] Hashes
4. [x] Checkver
5. [x] Autoupdate
6. [ ] Install (#59)
7. [ ] Format (#24)

Expected comment:

Thanks for contributing. Your PR did not pass requirements.

### Manifest 1

- [x] Description
- [ ] License
- [x] Hashes
- [x] Checkver
- [x] Autoupdate

### Manifest 2

- [x] Description
- [x] License
- [x] Hashes
- [x] Checkver
- [x] Autoupdate

Thanks for contributing. PR looks good.

Wait for review from human collaborators.

Implementation:

Best possible to temporary save into some hashtable:

@{name = 'Manifest1'; 'Checkver' = Passed; 'Properties': 'OK | list of failed properties'}

[Ash258]

~~Rework test-hash function to be reusable.~~

There is no need to do it this hard way. Keep it capturing streams and wait for binaries rework

[Ash258]

~~Resolve how to run tests in context of pull request files~~

DONE

[Ash258]

~~Try PR from forked repository~~

DONE

[Ash258]

~~Basic implementation is done.~~

~~Now comes more problematic things:~~

~~- Install check~~ ~~- Format check~~

Issue description

[Ash258]

~~When invoking from forked repository, context is on current repository not forked.~~

~~Investigate how to do it with push event~~

Rather do it with custom cloning forked repository

[Ash258]

~~Add more license checks:~~

~~- [x] If property exists~~ ~~- [ ] If it is valid spdx identifier~~ ~~- [ ] No? Is there URL?~~

~~md~~ ~~- [ ] License~~ ~~- [ ] Valid SPDX Identifier~~ ~~- [ ] Invalid + URL~~ ~~~~

~~This will require adding $Indent switch to create-new-checklist function~~

#21

[Ash258]

~~Add note about log:~~

~~Please see run log for pure result of all checks~~

[Ash258]

Forked repositories are now covered and working.

[Ash258]

Pull request comments with /verify are handled

[Ash258]

For first release this is enough.

Install and format will come in future versions