This repository has been archived by the owner on Apr 23, 2021. It is now read-only.

Consider renaming ExtractFrom/InjectInto to Source/Destination #27

Closed

slashmo opened this issue Jun 12, 2020 · 1 comment · Fixed by #55

Assignees

Labels

Milestone

Coding Period 1: ...

Owner

slashmo commented Jun 12, 2020

Via @tanner0101

The associated types InjectInto and ExtractFrom are prepositions, not nouns. Maybe something like Source / Destination would be better?

ktoso added the t:context label

Collaborator

ktoso commented Jun 29, 2020

As we adopt Extractor / Injectors these are likely to change, @slashmo investigating now.

ktoso assigned slashmo

ktoso added this to the Coding Period 1: NIO + Tracing milestone

slashmo mentioned this issue

Introduce Carrier, Injector, and Extractor types #55

Merged

slashmo closed this as completed in #55

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.