All organization members should be assignable as reviewer

[jpraet]

For PR's in public repos, all organization members should be assignable as reviewer.

[codecov-io]

Codecov Report

Merging #14262 (88a90e3) into master (154b23d) will increase coverage by 0.19%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #14262      +/-   ##
==========================================
+ Coverage   41.98%   42.18%   +0.19%     
==========================================
  Files         760      760              
  Lines       81199    81203       +4     
==========================================
+ Hits        34094    34254     +160     
+ Misses      41535    41360     -175     
- Partials     5570     5589      +19     

Impacted Files	Coverage Δ
modules/base/tool.go	94.25% <ø> (+19.61%)	⬆️
models/repo.go	52.57% <100.00%> (+0.16%)	⬆️
modules/util/timer.go	42.85% <0.00%> (-42.86%)	⬇️
models/repo_mirror.go	16.66% <0.00%> (-11.91%)	⬇️
modules/git/repo_base_nogogit.go	63.63% <0.00%> (-9.10%)	⬇️
modules/charset/charset.go	68.53% <0.00%> (-6.75%)	⬇️
services/pull/check.go	48.52% <0.00%> (-2.95%)	⬇️
modules/cron/tasks_basic.go	85.43% <0.00%> (-2.92%)	⬇️
modules/git/utils.go	77.77% <0.00%> (-2.78%)	⬇️
models/unit.go	46.57% <0.00%> (-2.74%)	⬇️
... and 22 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 99b7af6...88a90e3. Read the comment docs.

[zeripath]

Approving but please fix the comment on line 596 as per https://github.com/go-gitea/gitea/pull/14262/files#r566356479

[lunny]

Since the logic changed, I marked it as a break change.

[jpraet]

From my point of view I wouldn't call this a breaking change. Organization members were already allowed to leave reviews on PRs. This change just makes it possible to explicitly request an organization member's review.

[zeripath]

make lg-tm work