Document the process for having gem update MRs checked before merge.
When a new or updated gem is introduced to GitLab that involves native extensions, it should be tested by building omnibus on all our platforms before it gets merged into master.
The code review guidelines should have a note added about gems being compiled in all supported platforms: https://gitlab.com/gitlab-org/gitlab-foss/-/blob/master/doc/development/gemfile.md#review-the-new-dependency-for-quality
Currently the guidelines say:
If the project uses a C extension, consider requesting an additional review from a C or MRI domain expert. C extensions can greatly impact GitLab stability and performance.
What we actually want, is simply a CI build on all supported platforms.
Edited by DJ Mountney