search

bfenetworks / bfe

A modern layer 7 load balancer from baidu
https://www.bfe-networks.net
Apache License 2.0
6.08k stars 942 forks source link

staticcheck action not in ci job #1017

Open kwanhur opened 2 years ago

kwanhur commented 2 years ago

staticcheck had been used in bfe, but it doesn't exist in ci job steps.

Add step staticcheck on pull request and push develop, helps to improve quality of codes.

marketplace here, but it's under MIT license. Before using it, any license conflicts and CNCF policy limitations should take a care.

kwanhur commented 2 years ago

Another way, just add one step run with staticcheck binary. I think it hasn't any conflicts.

kwanhur commented 2 years ago

Advice adding staticcheck into pre-commit.