RedHatInsights / yggdrasil

GNU General Public License v3.0
21 stars 37 forks source link

ci: added golines lint checking #156

Closed DuckBoss closed 1 year ago

DuckBoss commented 1 year ago

This PR adds golines checking to new PRs and prevents merging to main if it detects that the code has not been formatted for golines.

Note: Not sure if there's a better way to do this, but If there's a way to automatically run golines on every PR during merging instead of requiring commits to comply to golines formatting then that might be a better solution. Of course that could introduce a potential for errors caused by golines to be merged into main so I'm not sure.