Skip to content

Conversation

@plyr4
Copy link
Contributor

@plyr4 plyr4 commented Aug 17, 2023

continuing the efforts started in #898 #922 and #923

this is prep to add support for OpenTelemetry tracing.

the reason being, otel can (for the most part) work out of the box if you pass around contexts properly.

@plyr4 plyr4 requested a review from a team as a code owner August 17, 2023 14:17
@plyr4 plyr4 changed the title enhance: add context to Repos chore: add context to repos functions Aug 17, 2023
@codecov
Copy link

codecov bot commented Aug 17, 2023

Codecov Report

Merging #930 (c794a50) into main (7dc577f) will increase coverage by 0.01%.
The diff coverage is 90.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #930      +/-   ##
==========================================
+ Coverage   71.07%   71.09%   +0.01%     
==========================================
  Files         304      304              
  Lines       12512    12520       +8     
==========================================
+ Hits         8893     8901       +8     
  Misses       3177     3177              
  Partials      442      442              
Files Changed Coverage Δ
api/build/create.go 0.00% <0.00%> (ø)
api/build/get_id.go 0.00% <0.00%> (ø)
api/build/restart.go 0.00% <0.00%> (ø)
database/repo/count.go 100.00% <100.00%> (ø)
database/repo/count_org.go 100.00% <100.00%> (ø)
database/repo/count_user.go 100.00% <100.00%> (ø)
database/repo/create.go 55.88% <100.00%> (ø)
database/repo/delete.go 100.00% <100.00%> (ø)
database/repo/get.go 90.00% <100.00%> (ø)
database/repo/get_org.go 91.17% <100.00%> (ø)
... and 10 more

Copy link
Contributor

@KellyMerrick KellyMerrick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

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.

4 participants