Fix repo listing

[qwerty287]

closes #2854

[codecov]

Codecov Report

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

Comparison is base (9d5ef11) 34.05% compared to head (1e9f882) 34.06%.
Report is 40 commits behind head on main.

Files	Patch %	Lines
pipeline/frontend/metadata/environment.go	0.00%	16 Missing ⚠️
server/api/repo.go	0.00%	11 Missing ⚠️
server/forge/gitlab/gitlab.go	50.00%	9 Missing ⚠️
server/store/datastore/xorm.go	0.00%	8 Missing ⚠️
server/api/login.go	0.00%	7 Missing ⚠️
shared/utils/slices.go	0.00%	7 Missing ⚠️
server/api/user.go	0.00%	5 Missing ⚠️
pipeline/frontend/yaml/compiler/cacher.go	0.00%	4 Missing ⚠️
server/forge/github/github.go	0.00%	4 Missing ⚠️
...erver/store/datastore/migration/027_link_to_url.go	40.00%	2 Missing and 1 partial ⚠️
... and 13 more

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2855   +/-   ##
=======================================
  Coverage   34.05%   34.06%           
=======================================
  Files         214      214           
  Lines       13779    13802   +23     
=======================================
+ Hits         4693     4702    +9     
- Misses       8714     8727   +13     
- Partials      372      373    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pat-s]

Thanks for the fix!

For such essential code parts: can we add a test? (must not necessarily go in here due to time pressure of v2, but it might get forgotten otherwise)

[qwerty287]

For such essential code parts: can we add a test? (must not necessarily go in here due to time pressure of v2, but it might get forgotten otherwise)

Sure, tests are always good