unit + import dagster dones't produce deprecation warnings
not sure why https://sourcegraph.com/github.com/dagster-io/dagster/-/blob/python_modules/dagster/dagster_tests/core_tests/test_import.py isn't working
If I understand correctly, this will push the deprecation warning from pipeline definition time to pipeline run time? Not a big problem, but if there's an easy way to surface a warning earlier, that could be nice.
@sandyryza that's right. the issue was caused by fs_intermediate_storage and mem_intermediate_storage in the dagster/__init__.py which call the IntermediateStorageDefinition path. So if we want to warn at definition time, the way i could think of is to pass in a flag like do_not_warn to IntermediateStorageDefinition so fs_intermediate_storage and mem_intermediate_storage won't fire deprecation warnings when import dagster. but i felt that was a bit hacky so i ended up moving it to pipeline run time warning