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

feat: add logging hook, rm logging from evaluation #289

Merged
merged 13 commits into from
Oct 23, 2024

Update README.md

81acdf6
Select commit
Loading
Failed to load commit list.
Merged

feat: add logging hook, rm logging from evaluation #289

Update README.md
81acdf6
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 23, 2024 in 0s

82.75% of diff hit (target 81.89%)

View this Pull Request on Codecov

82.75% of diff hit (target 81.89%)

Annotations

Check warning on line 371 in openfeature/event_executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

openfeature/event_executor.go#L371 

Added line #L371 was not covered by tests

Check warning on line 67 in openfeature/hooks/logging_hook.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

openfeature/hooks/logging_hook.go#L66-L67 

Added lines #L66 - L67 were not covered by tests

Check warning on line 77 in openfeature/hooks/logging_hook.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

openfeature/hooks/logging_hook.go#L76-L77 

Added lines #L76 - L77 were not covered by tests

Check warning on line 89 in openfeature/hooks/logging_hook.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

openfeature/hooks/logging_hook.go#L88-L89 

Added lines #L88 - L89 were not covered by tests
View more details on Codecov