quiet-afternoon-68940
03/28/2024, 9:32 PMEnvironmentDecorator has allow_multiple=False. Typically someone sets an @environment in code, and then someone else tries to deploy --with environment and wonders why their environment variables aren't set. We've settled on working around this by writing custom decorators that modify an existing @environment decorator, instead of using @environment. This works, except when people apply the custom decorator without an @environment decorator and wonder why their environment variables didn't get set. Metaflow's behavior here seems counter-intuitive to me. I would expect multiple environment decorators to have the same effect as applying a single decorator whose vars attribute was determined by merging the vars of the individual decorators (with some room for discussion about handling conflicts). Would there be any interest in a PR to implement something like this? Or is the current behavior desirable? Or is it too late to change without breaking people's existing code?