fix: treat nocase:true as always having magic

Because `a` can match either `A` or `a` in nocase mode, technically
almost any character is "magic".

minimatch.js

@@ -363,11 +363,7 @@ class Minimatch {
       // patterns with magic have been turned into regexps.
       var hit
       if (typeof p === 'string') {
-        if (options.nocase) {
-          hit = f.toLowerCase() === p.toLowerCase()
-        } else {
-          hit = f === p
-        }
+        hit = f === p
         this.debug('string match', p, f, hit)
       } else {
         hit = f.match(p)
@@ -430,7 +426,7 @@ class Minimatch {
     if (pattern === '') return ''
 
     let re = ''
-    let hasMagic = false
+    let hasMagic = !!options.nocase
     let escaping = false
     // ? => one single character
     const patternListStack = []

test/patterns.js

@@ -330,7 +330,7 @@ module.exports.regexps = [
   '/^(?:(?=.)a[^/]*?[^/]*?[^/]*?[^/]*?[^/]*?[^/]*?[^/]*?[^/]*?[^/][^/][^/][^/]*?[^/]*?[^/]*?[^/]*?[^/]*?[^/]*?[^/]*?)$/',
   '/^(?:\\[\\])$/',
   '/^(?:\\[abc)$/',
-  '/^(?:XYZ)$/i',
+  '/^(?:(?=.)XYZ)$/i',
   '/^(?:(?=.)ab[^/]*?)$/i',
   '/^(?:(?!\\.)(?=.)[ia][^/][ck])$/i',
   '/^(?:\\/(?!\\.)(?=.)[^/]*?|(?!\\.)(?=.)[^/]*?)$/',