[GitHub] [superset] ktmud commented on a change in pull request #13498: build: check potential db migration conflict for open PRs

[GitBox <gi...@apache.org>]

ktmud commented on a change in pull request #13498:
URL: https://github.com/apache/superset/pull/13498#discussion_r588812656



##########
File path: CONTRIBUTING.md
##########
@@ -1016,15 +1016,46 @@ To fix it:
    superset db heads
    ```
 
-   This should list two or more migration hashes.
+   This should list two or more migration hashes. E.g.
+
+   ```bash
+   1412ec1e5a7b (head)
+   67da9ef1ef9c (head)
+   ```
+
+2. Pick one of them as the parent revision, open the script for the other revision
+   and update `Revises` and `down_revision` to the new parent revision. E.g.:

Review comment:
       I'm also updating the contribution guide on how to avoid adding a new empty db migration merge file.




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