Add docstring and type annotation to merger.merge_dicts
ClosedPublic
Actions

Authored by • sandyryza on Aug 13 2021, 3:16 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Mar 25, 8:45 PM

	Unknown Object (File)
	Fri, Mar 10, 4:38 PM

	Unknown Object (File)
	Fri, Mar 10, 4:38 PM

	Unknown Object (File)
	Fri, Mar 10, 4:38 PM

	Unknown Object (File)
	Fri, Mar 10, 4:32 PM

	Unknown Object (File)
	Thu, Mar 9, 11:43 AM

	Unknown Object (File)
	Feb 14 2023, 9:58 AM

	Unknown Object (File)
	Feb 13 2023, 10:40 AM

View All 100 Files

Subscribers

None

Details

Reviewers

alangenfeld

Commits

R1:2073cf90bc41: Add docstring and type annotation to merger.merge_dicts

Test Plan

Diff Detail

Repository

R1 dagster

Branch

merger-docstring (branched from master)

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

• sandyryza created this revision.Aug 13 2021, 3:16 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 13 2021, 3:39 PM

Harbormaster failed remote builds in B34653: Diff 42870!

Harbormaster completed remote builds in B34657: Diff 42875.Aug 13 2021, 6:50 PM

• sandyryza requested review of this revision.Aug 13 2021, 6:50 PM

alangenfeld accepted this revision.Aug 24 2021, 4:15 PM

This revision is now accepted and ready to land.Aug 24 2021, 4:15 PM

Closed by commit R1:2073cf90bc41: Add docstring and type annotation to merger.merge_dicts (authored by • sandyryza). · Explain WhyAug 24 2021, 4:24 PM

This revision was automatically updated to reflect the committed changes.

• sandyryza added a commit: R1:2073cf90bc41: Add docstring and type annotation to merger.merge_dicts.

Revision Contents
Changeset List

Path

Size

python_modules/

dagster/

utils/

merger.py

17 lines

Diff 42875

View Options

python_modules/dagster/dagster/utils/merger.py

Add docstring and type annotation to merger.merge_dictsClosedPublicActions