Differential D5170

[easy] Fix deprecation warning and do not return value from hammer_pipeline
ClosedPublic
Actions

Authored by • schrockn on Nov 17 2020, 7:30 PM.

Tags

None

Subscribers

None

Details

Reviewers

alangenfeld
• nate
• sashank

Commits

R1:15de4f90b3d9: [easy] Fix deprecation warning and do not return value from hammer_pipeline

Summary

Don't return things from @pipeline

Test Plan

BK. Load dagit at root, see no warning

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• schrockn created this revision.Nov 17 2020, 7:30 PM

Harbormaster completed remote builds in B21278: Diff 25812.Nov 17 2020, 7:44 PM

• schrockn requested review of this revision.Nov 17 2020, 7:44 PM

alangenfeld accepted this revision.Nov 17 2020, 9:33 PM

This revision is now accepted and ready to land.Nov 17 2020, 9:33 PM

Closed by commit R1:15de4f90b3d9: [easy] Fix deprecation warning and do not return value from hammer_pipeline (authored by • schrockn). · Explain WhyNov 17 2020, 9:49 PM

This revision was automatically updated to reflect the committed changes.

• schrockn added a commit: R1:15de4f90b3d9: [easy] Fix deprecation warning and do not return value from hammer_pipeline.

Revision Contents
Changeset List

Path

Size

python_modules/

dagster-test/

dagster_test/

toys/

3 lines

Diff 25836

python_modules/dagster-test/dagster_test/toys/hammer.py

Loading...