Page MenuHomeDevCentral

Fix version regex
ClosedPublic

Authored by dereckson on Apr 22 2019, 08:03.
Tags
None
Referenced Files
F10818640: D2064.id5208.diff
Sun, Jul 27, 17:38
F10815845: D2064.id5206.diff
Sun, Jul 27, 15:11
F10806708: D2064.id5206.diff
Sun, Jul 27, 09:40
F10790157: D2064.id5207.diff
Sat, Jul 26, 22:45
Unknown Object (File)
Sat, Jul 26, 19:28
Unknown Object (File)
Fri, Jul 25, 16:03
Unknown Object (File)
Fri, Jul 25, 14:54
Unknown Object (File)
Mon, Jul 21, 05:29
Subscribers
None

Details

Summary

Allow to use version pinning with the multiline output
of shellcheck --version.

Test Plan

arc lint --everything

Diff Detail

Repository
rSCL ShellCheck linter for Arcanist
Lint
Lint Passed
Unit
Tests Passed
Branch
dereka/fix-version-check
Build Status
Buildable 3216
Build 3465: arc lint + arc unit

Event Timeline

dereckson created this revision.
This revision is now accepted and ready to land.Apr 22 2019, 08:04

Can't repro directly through arc lint interaction the issue.

Probably best to add a unit test.

Without the fix

$ arc unit --everything
   FAIL  ArcanistShellCheckLinterTestCase::testGetVersion
Assertion failed, expected 'false' (at ArcanistShellCheckLinterTestCase.php:9): The version can't be extracted from the binary output.

With the fix

$ arc unit
   PASS   35ms★ ArcanistShellCheckLinterTestCase::testGetVersion
This revision was automatically updated to reflect the committed changes.