From: Denton Liu Date: Thu, 26 Mar 2020 08:33:46 GMT Subject: Re: [PATCH 1/2] t/lib-credential.sh: drop shebang line Message-ID: <20200326083346.GA1891350@generichostname> In-Reply-To: <20200326082527.GA3611762@coredump.intra.peff.net> On Thu, Mar 26, 2020 at 04:25:27AM -0400, Jeff King wrote: > The purpose of lib-credential.sh is to be sourced into other test > scripts. It doesn't need a "#!/bin/sh" line, as running it directly > makes no sense. Nor does it serve any real filetype documentation > purpose, as the file is cleary named with a ".sh" extension. s/cleary/clearly/ Patch and message look obviously good otherwise.