mirror of
https://codeberg.org/davrot/forgejo.git
synced 2025-05-03 08:00:03 +02:00
00761a15d1
- Split `GetDiff` into two functions, `GetDiffSimple` and `GetDiffFull`. The former will do the bare minimum and really only get a Git diff while the latter does some extra stuff that's relevant for the frontend to show extra relevant.
- Use `GetDiffSimple` for API related calls, as they do not benefit nor are returning the extra information that `GetDiffFull` provides, this should show a measurable performance increase for API calls that returns commits and `/repos/{owner}/{repo}/pulls/{index}/files`.
- `GetDiffSimple` contains extra code comments about its interesting way to determine the before commit.
- Add unit tests to demonstrates that the logic for determining the before commit didn't change and the function still yields correct information.
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/7682
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
Co-authored-by: Gusted <postmaster@gusted.xyz>
Co-committed-by: Gusted <postmaster@gusted.xyz>
|
||
|---|---|---|
| .. | ||
| testdata/academic-module | ||
| csv.go | ||
| csv_test.go | ||
| gitdiff.go | ||
| gitdiff_test.go | ||
| highlightdiff.go | ||
| highlightdiff_test.go | ||
| main_test.go | ||