mistercrunch commented on issue #33162: URL: https://github.com/apache/superset/issues/33162#issuecomment-2863784553
Oh gotcha, I didn't look at the internal logic around dependency management, but normally we recommend installing the pinned dependencies (as we do in our Dockerfile). The pinned deps are the `.txt` here https://github.com/apache/superset/tree/master/requirements . From my understanding we never pointed to `marshmallow==4.0.0` in those files in `master` or any releases since would have failed our CI. You may want to alter your build process to install those pinned dependency to align with the libs we run our CI against and ship in our Docker. Normally, we would have caught this issue and produce a similar fix when our `dependabot` equivalent (`supersetbot`) would have tried to upgrade masrshmallow. But glad you caught and fix it! -- 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]
