chipsalliance / verible-linter-action

Automatic SystemVerilog linting in github actions with the help of Verible
Apache License 2.0
24 stars 11 forks source link

Replace double workflow setup with `pull_request_target` workflow trigger #31

Closed koluckirafal closed 1 year ago

koluckirafal commented 1 year ago

Double workflow setup makes it harder to notice problems with actual linting action, because it's not visible from pull request page. Changing workflow trigger to pull_request_target with different reference to be checked out allows for running linting process with elevated permissions for pull requests from forked repositories. Required permissions are explicitly listed to limit access to unused GitHub API endpoints.