Add shellcheck linter
ClosedPublic
Actions

Authored by dereckson on Nov 3 2017, 18:24.

Tags

Referenced Files

	Unknown Object (File)
	Sun, Sep 14, 15:06

	Unknown Object (File)
	Sat, Sep 13, 17:44

	Unknown Object (File)
	Tue, Sep 9, 23:40

	Unknown Object (File)
	Mon, Sep 8, 01:19

	Unknown Object (File)
	Sun, Sep 7, 16:15

	Unknown Object (File)
	Sat, Sep 6, 02:04

	Unknown Object (File)
	Fri, Sep 5, 03:36

	Unknown Object (File)
	Mon, Sep 1, 02:17

View All Files

Subscribers

Nasqueron Docker deployment squad

Details

Reviewers

dereckson

Commits

rDARC05c2c83f7a93: Add shellcheck linter

Summary

Setup similar to rOPSf637b1b858173

Test Plan

arc lint

Diff Detail

Repository

rDARC nasqueron/arcanist Docker image

Lint

No Lint Coverage

Unit

No Test Coverage

Branch

shellcheck (branched from master)

Build Status

Buildable 1829
Build 2077: arc lint + arc unit

Event Timeline

dereckson created this revision.Nov 3 2017, 18:24

Herald added a project: Docker images. · View Herald TranscriptNov 3 2017, 18:24

Herald added a subscriber: Nasqueron Docker deployment squad. · View Herald Transcript

Harbormaster completed remote builds in B1829: Diff 2971.Nov 3 2017, 18:24

Works fine:

$ cat > foo.sh
#!/bin/sh

foo $1
$ git add foo.sh 
$ arc lint
>>> Lint for foo.sh:


   Advice  (SC2086) SHELLCHECK
    Double quote to prevent globbing and word splitting.

               1 #!/bin/sh
               2 
    >>>        3 foo $1
                     ^
 OKAY  No lint warnings.