Consolidate ref checking logic for FE and BE
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
Background
The following discussion from !101930 (merged) should be addressed:
-
@mrincon started a discussion: (+1 comment) Suggestion (non-blocking): We have a very similar regex here: https://gitlab.com/gitlab-org/gitlab/blob/master/lib/api/validations/validators/git_ref.rb#L12
Maybe we can use the same list or cross link these with comment?
Problem to solve
-
We should investigate how to best consolidate the regex / checks we use for refs
: -
Move related utils to a !101930 (comment 1286465451) (if it still makes sense)
Edited by 🤖 GitLab Bot 🤖