bxsx commented on PR #42494: URL: https://github.com/apache/superset/pull/42494#issuecomment-5149008986
Hi @rusackas - yeah, I created that merge commit to re-trigger CI after the first run got cancelled, but I hadn't noticed that a new DB migration had landed on master in the meantime. Now fixed: rebased onto master and repointed 2d6ad72e4af6 at the current head (e7d93a524ff6) - pushed as 0d59fc11f715. Sorry for the delay - I was honestly burned out after all those agent reviews (here and on #42495), since most of them turned out to be false positives (this migration one being a notable exception). So I decided to take a break and wait for a human to take an interest in this PR (thanks!) rather than waste more time arguing with bots ;) I believe the PR is in good shape now. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
