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.

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

Add NoHTTP init mode for ephemeral tasks #16

Merged
merged 1 commit into from
Jul 13, 2023
Merged

Conversation

vearutop
Copy link
Member

No description provided.

@github-actions
Copy link

Lines Of Code

Language Files Lines Code Comments Blanks Complexity Bytes
Go 37 2215 (+7) 1612 (+5) 169 (+1) 434 (+1) 266 (+2) 54.5K (+149B)

@github-actions
Copy link

Go API Changes

# github.com/bool64/brick
## compatible changes
StartOptions.NoHTTP: added

# summary
Inferred base version: v0.2.0
Suggested version: v0.3.0

@github-actions
Copy link

Unit Test Coverage

total: (statements) 57.7%
changed lines: (statements) 0.0%, coverage is less than 90.0%, consider testing the changes more thoroughly

Coverage of changed lines
File Function Coverage
Total 0.0%
cmd.go 0.0%
cmd.go:33 Start 0.0%
Coverage diff with base branch

No changes in coverage.

@github-actions
Copy link

Benchmark Result

Benchmark diff with base branch
Module github.com/bool64/dev not found, downloading.
name          old time/op    new time/op    delta
CallerFunc-2    2.20µs ± 4%    2.17µs ± 2%   ~     (p=0.329 n=6+5)

name          old alloc/op   new alloc/op   delta
CallerFunc-2      264B ± 0%      264B ± 0%   ~     (all equal)

name          old allocs/op  new allocs/op  delta
CallerFunc-2      4.00 ± 0%      4.00 ± 0%   ~     (all equal)
Benchmark result
name          time/op
CallerFunc-2  2.17µs ± 2%

name          alloc/op
CallerFunc-2    264B ± 0%

name          allocs/op
CallerFunc-2    4.00 ± 0%

@codecov-commenter
Copy link

Codecov Report

Merging #16 (8dc75e7) into master (60a0668) will decrease coverage by 0.45%.
The diff coverage is 0.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   53.38%   52.94%   -0.45%     
==========================================
  Files          10       10              
  Lines         354      357       +3     
==========================================
  Hits          189      189              
- Misses        141      144       +3     
  Partials       24       24              
Flag Coverage Δ
unittests 52.94% <0.00%> (-0.45%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cmd.go 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@vearutop vearutop merged commit 6c5c169 into master Jul 13, 2023
6 of 7 checks passed
@vearutop vearutop deleted the photo-blog-contrib-pt2 branch July 13, 2023 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants