Page MenuHomeElementl

Typehints for MayHaveInstanceWeakref
ClosedPublic

Authored by max on Jul 2 2021, 9:37 PM.
Tags
None
Referenced Files
F2439114: D8696.id40913.diff
Tue, Aug 16, 6:59 PM
F2439113: D8696.id.diff
Tue, Aug 16, 6:59 PM
Unknown Object (File)
Mon, Aug 15, 8:21 PM
Unknown Object (File)
Mon, Aug 15, 8:02 AM
Unknown Object (File)
Sun, Aug 14, 10:36 PM
Unknown Object (File)
Sat, Aug 13, 11:21 AM
Unknown Object (File)
Sat, Aug 13, 12:11 AM
Unknown Object (File)
Fri, Aug 12, 10:01 AM
Subscribers
None

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.Jul 2 2021, 10:02 PM
Harbormaster failed remote builds in B33158: Diff 40873!
Harbormaster returned this revision to the author for changes because remote builds failed.Jul 2 2021, 10:52 PM
Harbormaster failed remote builds in B33170: Diff 40887!

irrelevant stochastic test failure in dagster pandas

pandas should be behaving better if you rebase

This revision is now accepted and ready to land.Jul 3 2021, 12:06 AM
This revision was landed with ongoing or failed builds.Jul 3 2021, 12:35 AM
This revision was automatically updated to reflect the committed changes.

on a diff that adds typehints its probably a good idea to wait for the mypy test suite to pass before landing

https://dagster.phacility.com/D8702 fixes that and one other mypy issue that snuck in elsewhere