refactor(define-macros-order): simplify beforeMacro detection logic

Use array.find() instead of a for loop with break statement for a more
idiomatic and readable approach to finding the macro name.

Co-authored-by: Flo Edelmann <git@flo-edelmann.de>

Author: kzhrk

lib/rules/define-macros-order.js

@@ -221,14 +221,9 @@ function create(context) {
    */
   function reportNotOnTop(macro, nodes, before) {
     // Determine if 'before' is a macro from the order list
-    let beforeMacro = null
-    for (const macroName of order) {
-      const macroNodes = macrosNodes.get(macroName) ?? []
-      if (macroNodes.includes(before)) {
-        beforeMacro = macroName
-        break
-      }
-    }
+    const beforeMacro = order.find((macroName) =>
+      (macrosNodes.get(macroName) ?? []).includes(before)
+    )
 
     // Decide which message to use
     const messageId = beforeMacro ? 'macrosUnordered' : 'macrosNotAtTop'