Page MenuHomeElementl

Combine RepositoryLocationHandle and RepositoryLocation
ClosedPublic

Authored by dgibson on Mar 29 2021, 3:13 AM.

Details

Summary

There's no good reason for these to be separate anymore now that RepositoryLocationOrigin exists for loading locations from a serializable format.

Test Plan

Integration

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Mar 29 2021, 3:32 AM
Harbormaster failed remote builds in B28090: Diff 34425!
Harbormaster returned this revision to the author for changes because remote builds failed.Mar 29 2021, 4:14 AM
Harbormaster failed remote builds in B28094: Diff 34430!
Harbormaster returned this revision to the author for changes because remote builds failed.Mar 29 2021, 2:21 PM
Harbormaster failed remote builds in B28095: Diff 34431!

Impressive refactor, this is much cleaner and I'm all for this.

This revision is now accepted and ready to land.Mar 29 2021, 5:39 PM
This revision was landed with ongoing or failed builds.Mar 29 2021, 6:47 PM
This revision was automatically updated to reflect the committed changes.