Page MenuHomeDevCentral
Differential D1118

Improve Arcanist shell scripts
ClosedPublic
Actions

Authored by dereckson on Oct 25 2017, 02:34.
Tags
None
Referenced Files
F35359411: D1118.id2853.diff
Thu, Jul 9, 18:37
F35354730: D1118.id2854.diff
Thu, Jul 9, 17:24
F35339378: D1118.diff
Thu, Jul 9, 13:56
Unknown Object (File)
Mon, Jul 6, 15:44
Unknown Object (File)
Fri, Jul 3, 18:28
Unknown Object (File)
Fri, Jul 3, 12:35
Unknown Object (File)
Fri, Jul 3, 12:33
Unknown Object (File)
Thu, Jul 2, 09:22
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rWK298b018a6ff3: Improve Arcanist shell scripts
Summary

Fixes suggested by shellcheck:

  • use $() instead of deprecated ``
  • avoid cat when pipelines can be used
  • quote variables
Test Plan

shellcheck

Diff Detail

Repository
rWK Wynter Kitro utilities
Lint
Lint Passed
SeverityLocationCodeMessage
Advicearcanist/arc-paste-file.sh:16SC2094SHELLCHECK
Advicearcanist/arc-paste-file.sh:16SC2094SHELLCHECK
Unit
No Test Coverage
Branch
shellcheck (branched from master)
Build Status
Buildable 1760
Build 2008: arc lint + arc unit

Event Timeline

dereckson created this revision.Oct 25 2017, 02:34
dereckson accepted this revision.Oct 25 2017, 02:35

We're sure it only reads.

This revision is now accepted and ready to land.Oct 25 2017, 02:35
Closed by commit rWK298b018a6ff3: Improve Arcanist shell scripts (authored by dereckson). · Explain WhyOct 25 2017, 02:35
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
arcanist/
8 lines
6 lines
8 lines

Diff 2853

View Options

arcanist/arc-paste-diff.sh

Loading...
View Options

arcanist/arc-paste-file.sh

Loading...
View Options

arcanist/arc-update-repo.sh

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator