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
F3568918: D4828.id.diff
Sat, Dec 28, 12:40 AM
F3568345: D4828.id.diff
Fri, Dec 27, 10:24 PM
Unknown Object (File)
Thu, Dec 26, 5:40 PM
Unknown Object (File)
Thu, Dec 19, 11:01 PM
Unknown Object (File)
Thu, Dec 19, 11:01 PM
Unknown Object (File)
Sun, Dec 15, 10:34 PM
Unknown Object (File)
Sun, Dec 15, 10:34 PM
Unknown Object (File)
Sun, Dec 15, 10:34 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