Modernize workflows

[mmorel-35]

Description

Ubuntu-20.04 is not supported by github anymore.
actions/checkout@v3 is also not documented in the README

Also uses actions/checkouts instead of git clones, will reduce the space taken as the reference is precised with only one commit being checked out.

I also used stepsecurity to define permissions appropriately with commit sha

See

• https://docs.github.com/en/actions/reference/runners/github-hosted-runners
• https://github.com/actions/checkout
• https://app.stepsecurity.io/secure-workflow

[wbpcode]

LGTM. Thanks.

[wbpcode]

ci of c++11 is not supported. But I think maybe you can disable it first. And create an issue for that. We can see whether we can fix it or just abandon c++11 in our new version.

[mmorel-35]

I believe it is better to drop support for version before c++17 as long as dependencies like protobuf require c++17, see
https://github.com/protocolbuffers/protobuf/blob/main/.bazelrc

[wbpcode]

I am OK to that. We won't add too much new features to cpp2sky. So, I think it's fine to support only c++17 in our new version. who won't the old version support still could use our old version.

[mmorel-35]

I have removed '-Werror' so it doesn't fail on warnings.

[mmorel-35]

Following https://stackoverflow.com/a/39194946 recommendation

[wbpcode]

@mmorel-35 sorry for the delay. There is no pr-reminder there Orz.

[mmorel-35]

@wbpcode ,
Can you try again ? If this is not working my next change is to update bazelversion as to my understanding, in v4 of bazel all warnings are threated are errors but the disable of those three shall do the job

[wbpcode]

CI still not happy.

warning: unknown warning option '-Wno-array-parameter'; did you mean '-Wno-unused-parameter'? [-Wunknown-warning-option]