Fix hash keys in #describe

muellerj · Jonas Mueller · commit bbe629ae037f · 2019-08-26T18:08:35.000+02:00
Currently, the symbolized key is checked for presence and then the value for the non-symbolized one is used. This works for Hashes with indifferent access (like in rails etc.) but fails for a simple example like

```ruby
Git.open(".").describe("HEAD", abbrev: 0)

=&gt; Git::GitExecuteError (git '--git-dir=/folder/to/repo/.git' '--work-tree=/folder/to/repo' describe '--abbrev=' 'HEAD'  2&gt;&amp;1:error: option `abbrev' expects a numerical value)
```

Changed all references to the symbolized version (since the method signature suggests `@param [{Symbol=&gt;Object}]`).

Signed-off-by: Jonas Mueller &lt;jonas-mueller@mailbox.org&gt;
diff --git a/lib/git/lib.rb b/lib/git/lib.rb
@@ -113,12 +113,12 @@ def describe(committish=nil, opts={})
       arr_opts << '--always' if opts[:always]
       arr_opts << '--exact-match' if opts[:exact_match] || opts[:"exact-match"]
 
-      arr_opts << '--dirty' if opts['dirty'] == true
-      arr_opts << "--dirty=#{opts['dirty']}" if opts['dirty'].is_a?(String)
+      arr_opts << '--dirty' if opts[:dirty] == true
+      arr_opts << "--dirty=#{opts[:dirty]}" if opts['dirty'].is_a?(String)
 
-      arr_opts << "--abbrev=#{opts['abbrev']}" if opts[:abbrev]
-      arr_opts << "--candidates=#{opts['candidates']}" if opts[:candidates]
-      arr_opts << "--match=#{opts['match']}" if opts[:match]
+      arr_opts << "--abbrev=#{opts[:abbrev]}" if opts[:abbrev]
+      arr_opts << "--candidates=#{opts[:candidates]}" if opts[:candidates]
+      arr_opts << "--match=#{opts[:match]}" if opts[:match]
 
       arr_opts << committish if committish
 
