kbase / dts

A data transfer service
https://kbase.github.io/dts/
MIT License
0 stars 0 forks source link

Code cleanup and reorganization #47

Closed jeff-cohere closed 7 months ago

jeff-cohere commented 7 months ago

This PR is an attempt to clean things up a bit. I doubt it's perfect, but it seems like a structural improvement. Highlights:

Closes #42 Closes #43 Closes #44 Closes #46

codecov[bot] commented 7 months ago

Codecov Report

Attention: 36 lines in your changes are missing coverage. Please review.

Comparison is base (1e7f10a) 62.72% compared to head (3c6d9fd) 61.40%. Report is 2 commits behind head on main.

:exclamation: Current head 3c6d9fd differs from pull request most recent head 8be84b8. Consider uploading reports for the commit 8be84b8 to get more accurate results

Files Patch % Lines
tasks/tasks.go 52.00% 11 Missing and 1 partial :warning:
databases/jdp/database.go 68.18% 7 Missing :warning:
databases/databases.go 16.66% 5 Missing :warning:
endpoints/local/endpoint.go 68.75% 5 Missing :warning:
endpoints/globus/endpoint.go 72.72% 3 Missing :warning:
services/prototype.go 72.72% 3 Missing :warning:
endpoints/endpoints.go 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #47 +/- ## ========================================== - Coverage 62.72% 61.40% -1.33% ========================================== Files 12 12 Lines 2015 2026 +11 ========================================== - Hits 1264 1244 -20 - Misses 575 603 +28 - Partials 176 179 +3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.