superset-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [incubator-superset] bkyryliuk commented on pull request #11110: fix: fix alembic migration
Date Tue, 29 Sep 2020 23:08:30 GMT

bkyryliuk commented on pull request #11110:
URL: https://github.com/apache/incubator-superset/pull/11110#issuecomment-701038518


   > Thanks for the quick fix!
   > 
   > I wonder if there's a way in the future to require PRs that touch the migrations directory
to be rebased before merging. We wouldn't want to do this for all PRs, but it might be nice
for these specifically
   
   I think we had a separate CI test for it could be that I just have a bad memory 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


Mime
View raw message