mergestat
mergestat copied to clipboard
Add server-side input validation for repo insertions
Related to this work: https://github.com/mergestat/fuse/pull/157 we should add server-side input validation and normalization when adding a new repo, to enforce things like https prefix
Codecov Report
Merging #162 (a398f73) into main (43799f5) will not change coverage. The diff coverage is
n/a.
@@ Coverage Diff @@
## main #162 +/- ##
=======================================
Coverage 68.34% 68.34%
=======================================
Files 36 36
Lines 1744 1744
=======================================
Hits 1192 1192
Misses 355 355
Partials 197 197
| Impacted Files | Coverage Δ | |
|---|---|---|
| extensions/internal/git/native/blame.go | 64.36% <ø> (ø) |
|
| extensions/internal/git/native/files.go | 63.33% <ø> (ø) |
|
| extensions/internal/git/native/stats.go | 61.73% <ø> (ø) |
|
| extensions/internal/github/org_repos.go | 75.34% <ø> (ø) |
|
| extensions/internal/github/repo_issues.go | 71.22% <ø> (ø) |
|
| extensions/internal/github/repo_prs.go | 75.14% <ø> (ø) |
|
| extensions/internal/github/stargazers.go | 71.56% <ø> (ø) |
|
| extensions/internal/github/starred_repos.go | 68.88% <ø> (ø) |
|
| extensions/internal/github/user_repos.go | 72.60% <ø> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 43799f5...a398f73. Read the comment docs.
Something to consider here is also repo normalization - for instance https://github.com/mergestat/mergestat vs https://github.com/mergestat/mergestat/.
Both are considered separate repos right now, when they should be recognized as the same