Add zizmor linter and resolve warnings

[lwbt]

I see you are using pre-commit. I suggest you add zizmor as another linter for GHA.

Why did you chose to use - id: actionlint-system instead of - id: actionlint?

[psadi]

Hey @lwbt

I see you are using pre-commit. I suggest you add zizmor as another linter for GHA.

What is the benefit over actionlint? Could you please help to share insights or any documentation ?

Why did you chose to use - id: actionlint-system instead of - id: actionlint?

Reffering docs - i had actionlint installed via homebrew and wanted to use that instead of of the default Go Toolchain installation.

[lwbt]

Hey @lwbt

Why did you chose to use - id: actionlint-system instead of - id: actionlint?

Reffering docs - i had actionlint installed via homebrew and wanted to use that instead of of the default Go Toolchain installation.

I see. However it's a bit counter intuitive when you have to go out and hunt for dependencies instead of letting pre-commit handle it for you. I'm not using homebrew on Linux, but other tools like mise, eget, flatpak and more. It can certainly cover all needs, but it can be overwhelming, which is why I prefer to configure my pre-commit to decouple from the system and manage everything independently.

But it's your choice.

Is anything else required to be able to merge?

[psadi]

thanks for your insights, PR LGTM!

[psadi]

There is a later version v0.8.0 available, I will bump to latest version separately.