Skip to content

airfame-http-netty: Safe-guard for double start/stop#2925

Merged
xerial merged 1 commit intomasterfrom
double-start-prevention
May 4, 2023
Merged

airfame-http-netty: Safe-guard for double start/stop#2925
xerial merged 1 commit intomasterfrom
double-start-prevention

Conversation

@xerial
Copy link
Copy Markdown
Member

@xerial xerial commented May 4, 2023

No description provided.

@xerial xerial added the kaizen code improvement without changing user-facing functionality label May 4, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented May 4, 2023

Codecov Report

Merging #2925 (0cb95a8) into master (f6280d0) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2925   +/-   ##
=======================================
  Coverage   82.78%   82.79%           
=======================================
  Files         345      345           
  Lines       14551    14559    +8     
  Branches     2379     2401   +22     
=======================================
+ Hits        12046    12054    +8     
  Misses       2505     2505           
Impacted Files Coverage Δ
.../scala/wvlet/airframe/http/netty/NettyServer.scala 92.00% <100.00%> (+0.95%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f6280d0...0cb95a8. Read the comment docs.

@xerial xerial merged commit d808ee1 into master May 4, 2023
@xerial xerial deleted the double-start-prevention branch May 4, 2023 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kaizen code improvement without changing user-facing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant