8000 publish: Add ContextWithAppendDeferFunc by molon · Pull Request #605 · qor5/admin · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

publish: Add ContextWithAppendDeferFunc #605

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.

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

Merged
merged 1 commit into from
Sep 26, 2024
Merged

publish: Add ContextWithAppendDeferFunc #605

merged 1 commit into from
Sep 26, 2024

Conversation

molon
Copy link
Contributor
@molon molon commented Sep 26, 2024

No description provided.

@molon molon requested a review 8000 from zhangshanwen September 26, 2024 08:35
Copy link
codecov bot commented Sep 26, 2024

Codecov Report

Attention: Patch coverage is 66.66667% with 8 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
publish/list_publish_builder.go 40.00% 2 Missing and 1 partial ⚠️
publish/utils.go 72.72% 3 Missing ⚠️
example/cmd/publisher/main.go 0.00% 1 Missing ⚠️
publish/builder.go 0.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
publish/schedule_publish_builder.go 63.04% <100.00%> (+6.52%) ⬆️
example/cmd/publisher/main.go 0.00% <0.00%> (ø)
publish/builder.go 81.84% <0.00%> (ø)
publish/list_publish_builder.go 60.39% <40.00%> (+0.29%) ⬆️
publish/utils.go 50.87% <72.72%> (+6.87%) ⬆️

Copy link
deepsource-io bot commented Sep 26, 2024

Here's the code health analysis summary for commits 7ad4af3..dbb708b. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Test coverage LogoTest coverage✅ SuccessView Check ↗
DeepSource Go LogoGo✅ Success
🎯 17 occurences resolved
View Check ↗

Code Coverage Report

MetricAggregateGo
Composite Coverage40.6% (up 0.1% from main)40.6% (up 0.1% from main)
Line Coverage40.6% (up 0.1% from main)40.6% (up 0.1% from main)

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@molon molon merged commit 93b4fce into main Sep 26, 2024
11 checks passed
@molon molon deleted the publish-run-defer branch September 26, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0