Vidar Holen
|
5cf2c00ff7
|
Warn about defining and using an alias in a single command (fixes #1807)
|
2020-05-25 23:24:33 -07:00 |
|
Vidar Holen
|
a08ad3bee9
|
Count $# as an argument reference in SC2120
|
2020-05-25 23:24:33 -07:00 |
|
Vidar Holen
|
d6adbfde78
|
Improve SC2259/60/61 messages
|
2020-05-03 21:46:16 -07:00 |
|
Vidar Holen
|
2030b83607
|
Warn about duplicate uses of stdin/out/err
|
2020-05-03 11:54:25 -07:00 |
|
Vidar Holen
|
73cc11fd0a
|
Merge pull request #1901 from josephcsible/bracedstring
Mostly get rid of bracedString
|
2020-04-12 15:14:50 -07:00 |
|
Joseph C. Sible
|
b58bb4ba9d
|
Move bracedString to be local to its last use site
|
2020-04-11 19:24:11 -04:00 |
|
Joseph C. Sible
|
999b7e2596
|
Get rid of bracedString everywhere it's easy to
|
2020-04-11 19:24:11 -04:00 |
|
Joseph C. Sible
|
8a7497c4f0
|
Simplify checkVariableBraces
|
2020-04-11 19:23:13 -04:00 |
|
Joseph C. Sible
|
322842b57e
|
Remove unnecessary monadicity from wordToPseudoGlob
|
2020-04-05 19:29:40 -04:00 |
|
Vidar Holen
|
01f4423465
|
Disable SC2257 about > $((i=42)) for Dash
|
2020-04-05 11:38:22 -07:00 |
|
Vidar Holen
|
bd717c9d1b
|
Don't warn about [ 0 -ne $FOO ] || [ 0 -ne $BAR ] (fixes #1891)
|
2020-04-01 22:09:00 -07:00 |
|
Vidar Holen
|
da0931740f
|
Merge pull request #1876 from fork-graveyard/master
recognize `: ${parameter=word}` as assignment
|
2020-04-01 18:52:53 -07:00 |
|
Vidar Holen
|
c60323fb25
|
Merge pull request #1873 from josephcsible/checkwhilereadpitfalls
Simplify checkWhileReadPitfalls
|
2020-03-31 19:10:53 -07:00 |
|
girst
|
7a5e261d03
|
recognize : ${parameter=word} as assignment
|
2020-03-16 23:04:54 +01:00 |
|
Joseph C. Sible
|
9d5363377e
|
Simplify checkWhileReadPitfalls
* Clean up usage of not
* Use a case match instead of sequence_ and a do block
|
2020-03-16 00:14:22 -04:00 |
|
Joseph C. Sible
|
86d470c74f
|
Simplify checkForInQuoted
* Avoid some unnecessary fmaps
* Reuse an identical pattern-match for two guards
* Apply De Morgan's law
* Use forM_ to avoid an unnecessary where
|
2020-03-15 16:05:55 -04:00 |
|
Vidar Holen
|
a57f6d2886
|
Improve detection of for loops with single values
|
2020-03-15 11:30:56 -07:00 |
|
Vidar Holen
|
c43b19f897
|
Make SC2095 (ssh in while read loops) more robust and suggest fixes
|
2020-03-14 21:15:47 -07:00 |
|
Austin English
|
741d499b3d
|
src/ShellCheck/Analytics.hs: suggest using a shell directive for SC2148
|
2020-03-07 19:23:35 -06:00 |
|
Vidar Holen
|
7b998239af
|
SC2257: Warn when changing arithmetic variables in redirections
|
2020-02-17 18:16:57 -08:00 |
|
Vidar Holen
|
4c9210af79
|
Inspect 'alias' commands for referenced variables (Fixes #1832)
|
2020-02-17 14:20:21 -08:00 |
|
Vidar Holen
|
c2d67c15f8
|
Merge pull request #1802 from szydell/master
SC2016, repair false error for M language parser
|
2020-02-10 18:25:28 -08:00 |
|
Vidar Holen
|
6043deb8f2
|
Merge pull request #1824 from josephcsible/patch-1
Simplify literalEquals
|
2020-02-10 18:20:05 -08:00 |
|
Vidar Holen
|
d0beac6d0b
|
Merge pull request #1826 from josephcsible/nofromjust
Use the Identity monad to avoid unnecessary uses of fromJust
|
2020-02-10 18:05:36 -08:00 |
|
Vidar Holen
|
b88b253cad
|
Merge pull request #1827 from josephcsible/nofromjust2
Remove more unnecessary uses of fromJust
|
2020-02-10 18:01:38 -08:00 |
|
Joseph C. Sible
|
6d06103cab
|
Remove unnecessary uses of head
|
2020-02-09 23:18:09 -05:00 |
|
Joseph C. Sible
|
ea24e25efd
|
Use Map.member instead of isJust and Map.lookup
|
2020-02-09 23:18:09 -05:00 |
|
Joseph C. Sible
|
8f0448133c
|
Use isNothing instead of reimplementing it
|
2020-02-09 23:18:08 -05:00 |
|
Joseph C. Sible
|
7fc9496320
|
Use forM_ instead of reimplementing it
|
2020-02-09 23:18:08 -05:00 |
|
Joseph C. Sible
|
962fad038c
|
Avoid a zip that breaks fusion
|
2020-02-09 23:18:08 -05:00 |
|
Joseph C. Sible
|
a223a7a5a5
|
Remove unnecessary fromMaybes
|
2020-02-09 23:18:08 -05:00 |
|
Joseph C. Sible
|
8e9290badb
|
Do toLower earlier
|
2020-02-09 23:17:53 -05:00 |
|
Joseph C. Sible
|
292b0840d9
|
Simplify a double negative
|
2020-02-09 23:17:53 -05:00 |
|
Joseph C. Sible
|
43c24cf79c
|
Use Map.! instead of reimplementing it
|
2020-02-09 23:17:53 -05:00 |
|
Joseph C. Sible
|
21ad4196db
|
Simplify findFunction
|
2020-02-09 23:17:53 -05:00 |
|
Joseph C. Sible
|
172aa7c4fc
|
Avoid unnecessary use of when and unless
|
2020-02-09 23:17:53 -05:00 |
|
Joseph C. Sible
|
c290eace54
|
Inline an uncurry
|
2020-02-09 23:17:53 -05:00 |
|
Joseph C. Sible
|
a6efd02807
|
Simplify <> for SpaceStatus
|
2020-02-09 23:17:53 -05:00 |
|
Joseph C. Sible
|
057cc714b3
|
Simplify matchToken
|
2020-02-09 23:17:52 -05:00 |
|
Joseph C. Sible
|
0e00249eae
|
Use void instead of do and return ()
|
2020-02-09 23:17:52 -05:00 |
|
Joseph C. Sible
|
0ca50159ec
|
Use head instead of reimplementing it
Normally I wouldn't use head, but this code is partial anyway.
|
2020-02-09 23:17:52 -05:00 |
|
Joseph C. Sible
|
7e6a556ef1
|
Get rid of potentially
This already exists as sequence_.
|
2020-02-09 23:17:52 -05:00 |
|
Joseph C. Sible
|
4bfe6496d9
|
Simplify check and checkTranslatedStringVariable
Avoid the "potentially" and "Maybe" business, and just use regular guards.
|
2020-02-09 23:17:52 -05:00 |
|
Joseph C. Sible
|
ffbbfcfe25
|
Use mapM_ and sequence_ instead of reimplementing them
|
2020-02-09 23:17:52 -05:00 |
|
Joseph C. Sible
|
cc424bac11
|
Use find instead of take 1 and filter
|
2020-02-09 23:17:52 -05:00 |
|
Joseph C. Sible
|
cb01cbf7eb
|
Use mapM instead of implementing a slower version of it
|
2020-02-09 23:17:52 -05:00 |
|
Joseph C. Sible
|
1e32139f66
|
Replace mapMaybe and concatMap with list comprehensions
|
2020-02-09 23:17:52 -05:00 |
|
Joseph C. Sible
|
4d92a2e15c
|
Add getLiteralStringDef and simplify with it
|
2020-02-09 21:36:38 -05:00 |
|
Joseph C. Sible
|
f8648e5465
|
Switch getLiteralStringExt to Identity where it can never be Nothing
|
2020-02-09 21:26:42 -05:00 |
|
Joseph C. Sible
|
4fd8de058b
|
Remove more unnecessary uses of fromJust
|
2020-02-08 23:48:36 -05:00 |
|