Skip to content
New issue

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

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

Already on GitHub? # to your account

remove unnecessary call to time.Now() when sending a packet #2911

Merged
merged 1 commit into from
Dec 6, 2020

Conversation

marten-seemann
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Nov 29, 2020

Codecov Report

Merging #2911 (ca671ac) into master (5048006) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2911   +/-   ##
=======================================
  Coverage   86.24%   86.24%           
=======================================
  Files         133      133           
  Lines        9132     9132           
=======================================
  Hits         7875     7875           
  Misses        909      909           
  Partials      348      348           
Impacted Files Coverage Δ
session.go 79.41% <100.00%> (ø)

Continue to review full report at Codecov.

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

@marten-seemann marten-seemann force-pushed the replace-unnecessary-time-now-call branch from 93eb8c6 to 902a24f Compare December 5, 2020 16:15
@marten-seemann marten-seemann force-pushed the replace-unnecessary-time-now-call branch from 902a24f to ca671ac Compare December 6, 2020 02:25
@marten-seemann marten-seemann merged commit df38b0a into master Dec 6, 2020
@marten-seemann marten-seemann deleted the replace-unnecessary-time-now-call branch December 6, 2020 02:47
@aschmahmann aschmahmann mentioned this pull request May 14, 2021
71 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants