Skip to content
This repository has been archived by the owner on Apr 17, 2024. It is now read-only.

refactor: clock, rank, score, team route #171

Merged
merged 19 commits into from
Nov 12, 2021
Merged

refactor: clock, rank, score, team route #171

merged 19 commits into from
Nov 12, 2021

Conversation

wuhan005
Copy link
Contributor

@wuhan005 wuhan005 commented Oct 5, 2021

No description provided.

@codecov-commenter
Copy link

Codecov Report

Merging #171 (2f226e9) into master (e7e7fcb) will decrease coverage by 10.82%.
The diff coverage is 45.17%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #171       +/-   ##
===========================================
- Coverage   64.66%   53.84%   -10.83%     
===========================================
  Files          20       28        +8     
  Lines        1064     1653      +589     
===========================================
+ Hits          688      890      +202     
- Misses        323      690      +367     
- Partials       53       73       +20     
Flag Coverage Δ
unittests 53.84% <45.17%> (-10.83%) ⬇️

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

Impacted Files Coverage Δ
internal/clock/process.go 0.00% <0.00%> (ø)
internal/conf/test.go 0.00% <0.00%> (ø)
internal/db/db.go 0.00% <0.00%> (ø)
internal/route/general.go 5.00% <0.00%> (-5.00%) ⬇️
internal/route/manager.go 50.00% <0.00%> (+13.63%) ⬆️
internal/db/score.go 6.97% <6.97%> (ø)
internal/route/auth.go 15.38% <15.38%> (ø)
internal/route/team.go 51.63% <32.07%> (-19.62%) ⬇️
internal/route/challenge.go 68.00% <50.00%> (-2.79%) ⬇️
internal/clock/clock.go 52.00% <52.00%> (ø)
... and 10 more

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 e7e7fcb...2f226e9. Read the comment docs.

@wuhan005 wuhan005 merged commit 32f2ef4 into master Nov 12, 2021
@wuhan005 wuhan005 deleted the wh/refactor/route branch November 12, 2021 17:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants