search

DSACMS / metrics

Experimentations in Open Source Repository Metrics
https://dsacms.github.io/metrics/
Other
6 stars 3 forks source link

Add First Version of Repolinter Action #123

Closed IsaacMilarky closed 5 months ago

IsaacMilarky commented 5 months ago

Add First Version of Repolinter Action

Problem

Repolinter is not currently run automatically as a check GH action.

Solution

Use repolinter-action to run repolinter as an action. This action is using the tier 3 repolinter.json that was created. This is because if I remember correctly Metrics is a tier 3 project.

The action will create an issue on the project to give remediation steps to follow tier 3 project requirements.

IsaacMilarky commented 5 months ago

There seems to be some problems with the format of the repolinter.json file. I will look into this but shouldn't merge in the meantime. 

 Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Error: A fatal error was thrown.
Error: Error: Configuration validation failed with error Configuration validation failed with errors: 
    configuration.rules['maintainers-file-exists'] should have required property 'rule'
    at Object.rejected (/repolinter-action/webpack:/repolinter-action/lib/getConfig.js:109:1)
    at Generator.next (<anonymous>)
    at fulfilled (/repolinter-action/webpack:/repolinter-action/lib/getConfig.js:25:1)
IsaacMilarky commented 5 months ago

Ready to merge now. @natalialuzuriaga @decause-gov

IsaacMilarky commented 5 months ago

Since we are unable to extend from the Tier 2 repolinter.json file, I noted a couple of checks whose levels need to be changed to match Tier 3 requirements

Oh my bad I forgot to merge the Tier 3 changes into the JSON. Will fix