Page MenuHomePhabricator

[CI] Add “Install ShellCheck” step to ShellCheck GitHub Actions workflow
ClosedPublic

Authored by abosh on Aug 12 2022, 8:42 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 24, 3:37 PM
Unknown Object (File)
Tue, Sep 24, 3:37 PM
Unknown Object (File)
Tue, Sep 24, 3:37 PM
Unknown Object (File)
Tue, Sep 24, 3:37 PM
Unknown Object (File)
Fri, Sep 13, 2:55 AM
Unknown Object (File)
Sun, Sep 1, 12:00 PM
Unknown Object (File)
Aug 28 2024, 6:06 PM
Unknown Object (File)
Aug 21 2024, 5:30 PM
Subscribers

Details

Summary

See this comment left by me for more context, specifically this part:

Also, we could install ShellCheck through apt-get on GitHub Actions to keep it consistent with Buildkite. To do this, we'd just have to revert the changes made in D4756.

On Buildkite, we install ShellCheck through apt-get, so this diff keeps Buildkite and GitHub Actions CI symmetric by installing ShellCheck through the same source.

Test Plan

See what happens on GitHub Actions.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable