Skip to content
Snippets Groups Projects

Create a push rule to check branch name

All threads resolved!

What does this MR do?

It creates a new push rule to check the branch name

Are there points in the code the reviewer needs to double check?

Don't think so, but it is my first contribution, so everything :)

Why was this MR needed?

At work we heavily use Gitlab and Gitlab CI (\o/) and we have a strictly policy for branch names (actually, for the beginning of the name) because we have different Gitlab CI jobs (feature / hotfix / docker / android etc etc) and I am tired to have to remember which names are allowed and which not

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Fixes #2293 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • @rpadovani I had a few suggestions, please also update the image at doc/push_rules/push_rules.png.

  • @rymai thanks for your review and the patience with my beginner's errors.

    I fixed what you indicated and updated the image as well :-) Let me know if there is something else needed.

  • username-removed-16048 resolved all discussions

    resolved all discussions

  • changed milestone to %9.3

  • thanks for your review and the patience with my beginner's errors.

    @rpadovani That was really good already!

    Thanks, looks good to me! :heart:

  • username-removed-128633 approved this merge request

    approved this merge request

  • mentioned in commit eae6778a

  • Collen mentioned in issue #608 (closed)

    mentioned in issue #608 (closed)

  • Please register or sign in to reply
    Loading