Add license checker

[Fokko]

To make sure that we have the right headers 👍

[zhjwpku]

Do we need a empty line in the end here?

[wgtmac]

Should we consider the apache/skywalking-eyes@main Github action which Apache ORC currently uses: https://github.com/apache/orc/blob/main/.github/workflows/build_and_test.yml#L178-L189

It looks simpler to use IMHO.

[Fokko]

Yes, that looks much simpler indeed. Question, do you know why it needs the Github token?

[raulcd]

It seems required to comment on the PR:
https://github.com/apache/skywalking-eyes/blob/3ea9df11bb3a5a85665377d1fd10c02edecf2c40/pkg/review/header.go#L62-L64

[Fokko]

That's fancy, let me change to Skywalking!

[Fokko]

Done 👍 Tnx @wgtmac and @raulcd

[wgtmac]

Thanks @Fokko and @raulcd!

[zhjwpku]

add a newline to the end of a file.

[raulcd]

Thanks @Fokko . Just the minor nit of the removal of the whitespace

[Fokko]

Thanks @raulcd @zhjwpku Let's add pre-commit so we can add the yaml format hook to avoid manual formatting of yaml. Machines are much better at that :) Anyone interested in adding that?

[zhjwpku]

Thanks @raulcd @zhjwpku Let's add pre-commit so we can add the yaml format hook to avoid manual formatting of yaml. Machines are much better at that :) Anyone interested in adding that?

I can give it a look, thanks.

[raulcd]

Thanks @zhjwpku , I've created: #15