Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ZIO logging read context from Has, provide embedded Logger #225

Merged
merged 1 commit into from
Apr 12, 2020
Merged

ZIO logging read context from Has, provide embedded Logger #225

merged 1 commit into from
Apr 12, 2020

Conversation

Odomontois
Copy link
Member

@Odomontois Odomontois commented Apr 10, 2020 •
edited
Loading

Basically it gives some interop with zio.Has.
One can now easely define set of components, that should be logged during each operation.
Also define an embedded instance Loggable[URIO[R, *]] where R <: Has[Loggable[URIO[R1, *]]

@Odomontois Odomontois force-pushed the zio-logs branch 3 times, most recently from 9983d4c to a024d1b Compare April 10, 2020 11:29
@Odomontois Odomontois merged commit 7a3de02 into master Apr 12, 2020
@Odomontois Odomontois added the enhancement New feature or request label Apr 12, 2020
@catostrophe catostrophe deleted the zio-logs branch May 13, 2020 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@catostrophe catostrophe Awaiting requested review from catostrophe catostrophe is a code owner

@danslapman danslapman Awaiting requested review from danslapman danslapman is a code owner

@Z1kkurat Z1kkurat Awaiting requested review from Z1kkurat

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Odomontois