From 3ebc77ddba9b76de121e9164fbdf13ffd7baf3b7 Mon Sep 17 00:00:00 2001
From: koalaman <vidar@vidarholen.net>
Date: Tue, 11 Mar 2014 11:04:39 -0700
Subject: [PATCH] Created SC2051 (markdown)

---
 SC2051.md | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)
 create mode 100644 SC2051.md

diff --git a/SC2051.md b/SC2051.md
new file mode 100644
index 0000000..6c65590
--- /dev/null
+++ b/SC2051.md
@@ -0,0 +1,25 @@
+## Bash doesn't support variables in brace expansions.
+
+### Problematic code:
+
+    for i in {1..$n}
+    do
+      echo "$i"
+    done
+
+### Correct code:
+
+    for ((i=0; i<n; i++))
+    do
+      echo "$i"
+    done
+
+### Rationale:
+
+In Bash, brace expansion happens before variable expansion. This means that brace expansion will never account for variables.
+
+Use an arithmetic for loop instead.
+
+### Contraindications
+
+None (if you're writing for e.g. zsh, make sure the shebang indicates this so shellcheck won't warn)
\ No newline at end of file