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)
	Thu, May 11, 4:36 PM

	Unknown Object (File)
	Tue, May 9, 11:32 AM

	Unknown Object (File)
	May 3 2023, 9:31 PM

	Unknown Object (File)
	May 2 2023, 6:22 PM

	Unknown Object (File)
	Apr 30 2023, 12:35 PM

	Unknown Object (File)
	Apr 28 2023, 11:36 PM

	Unknown Object (File)
	Apr 28 2023, 1:28 AM

	Unknown Object (File)
	Apr 16 2023, 2:07 PM

View All 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

13 lines

Diff 42870

View Options

python_modules/dagster/dagster/utils/merger.py

Add docstring and type annotation to merger.merge_dictsClosedPublicActions