Conversation
|
test failure on macOS 10.15 test failure on macOS 10.14 test failure on macOS 10.13 These 3 are covered in #56539 new issues
|
|
do I still need to bump the revision in |
|
@jnozsc, another thing I just realized is if you checked the previous PRs, we should have two separate commits:
But we can totally do it after fixing the stuff that we can fix. |
|
@chenrui333 That's exactly what I am still confused. in previous PR #58115 , |
It's just deprecated. It isn't removed so people can and will still install it. And that install should be working correctly |
|
ok, thanks for clarifying this |
|
cancelled the build for now, and we can have a final run after fixing the stuff that we can fix. And then merging this patch upgrade. |
|
rebase from |
|
I am also inclined to do the merge and rebase to fix each formula (it is little bit costly to have golang runs) |
|
the test on 10.15 takes too much time, let's focus on 10.14 and 10.13 first tests fail on 10.14 tests fails on 10.13 the revision bump on what we have now
|
|
Emm... Have these formula been fixed? |
|
the test on |
nope, unless there is something very wrong happening. We can also cross map how much time it should run before. |
|
it took 13 hours on previous |
|
Two days over the previous release is indeed too long ( |
|
update the table above by adding a Due to the number of installation, I would like to suggest to rebase from |
|
trigger the test again by rebasing on |
|
trigger the test again by rebasing on |
|
I submit a ticket for |
|
|
|
I have acknowledged the There should be no new issues introduced, except For rest of these issues, they are being tracked in #56539 , basically most of them are pending on an upstream release |
|
10.15 10.14 10.13 summary
I suggest a merge and handle the leftover in #56539 |
|
Sounds good with me. Going for the merge now. |
|
Just 🚢 it. |
Created with
brew bump-formula-pr.