lua: fix type issues

[samford]

• Have you followed the guidelines for contributing?
• Have you ensured that your commits follow the commit style guide?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

---

As seen in #142161, the build for lua is currently failing due to some type issues.

TypeError: Parameter 'new_value': Expected type T.any(Pathname, String), got type NilClass
Caller: /opt/homebrew/Library/Taps/homebrew/homebrew-core/Formula/l/lua.rb:58
Definition: /opt/homebrew/Library/Homebrew/utils/string_inreplace_extension.rb:49

TypeError: Parameter 'new_value': Expected type T.any(Pathname, String), got type NilClass
Caller: /opt/homebrew/Library/Taps/homebrew/homebrew-core/Formula/l/lua.rb:59
Definition: /opt/homebrew/Library/Homebrew/utils/string_inreplace_extension.rb:49

With the two above, ENV.cflags and ENV.ldflags are nil. I'm on Sonoma, so I'm not sure if this is Sonoma-specific or happens on other macOS versions as well. If this should be handled in a different way than adding a conditional, feel free to suggest changes, push a commit, etc.

TypeError: Parameter 'version': Expected type T.nilable(T.any(Integer, String)), got type Version with value #<Version:0x000000015201e34....4", @detected_from_url=false>
Caller: /opt/homebrew/Library/Taps/homebrew/homebrew-core/Formula/l/lua.rb:105
Definition: /opt/homebrew/Library/Homebrew/formula.rb:1727

With this last one, we simply need to call to_s on version.major_minor there (the others are interpolated into strings).

[SMillerDev]

Team work makes the dream work, thanks @samford !

[fxcoudert]

I'd say we can merge without dependencies testing, at this point.

[samford]

I'd say we can merge without dependencies testing, at this point.

Oh shoot, I forgot to add the CI-skip-dependents and CI-skip-recursive-dependents labels. If we want to skip dependents, we could add those labels and start a new run (since lua builds really quickly).