Evan Rusackas
Evan Rusackas
@thedeeppp have you made any progress on this? Or can you confirm that it's still an issue in Superset 3.1? If not, this issue is at risk of being closed...
Let's try to get the linked PR across the finish line! Any updates/context or reviews/input welcomed!
Still hoping for a PR touch-up 🤞 Anyone is welcome to adopt it, if the author doesn't patch it.
Looks like the [pre-commit hooks](https://superset.apache.org/docs/contributing/hooks-and-linting/) also need to be run to make the linters happy.
Running CI... fingers crossed!
@nsivarajan thank you for your patience in the meantime... good to see that CI is passing and there's no conflicts still, at least! :D
Chart.yaml needs another bump, and you can run `pre-commit run --all-files` to trigger the `helm docs` job that'll make this mergeable. Installation instructions here: [pre-commit run --all-files](https://superset.apache.org/docs/contributing/hooks-and-linting/)
re-running that test, just in case.
Copilot's suggestions, FWIW: 1. Migration Failure Fix Update the migration script to handle the foreign key constraint before dropping the column. Modify the script to drop the foreign key first...
Resolved the conflict and currently re-running CI. Hopefully we can get this merged, pronto!