-
Notifications
You must be signed in to change notification settings - Fork 29
Description
Thanks for maintaining this GitHub Action - it's been very helpful!
I have a formula (see here) where the tarball url does not contain the periods from the version string. For example, the package version is 8.307 and the tarball url contains "8307". There are other formulae like this in homebrew-core (see here).
Running brew bump-formula-pr with only the --version argument attempts to update the URL using string substitution. Since the old url does not contain the old version string, it's unable to perform the substitution and so it cannot infer the new url. It raises the error You need to bump this formula manually since the new URL and old URL are both <old_url>.
Normally, the solution to this issue is to specify the --url argument. The difficulty for the GitHub Action is that we can't retrieve the new url from the livecheck command, so we can't automate this AFAIK.
Perhaps this underlying assumption should be documented? (i.e., URL must include version string exactly)