Skip to content
Snippets Groups Projects

Limit level of nesting for groups

Merged Dmytro Zaporozhets (DZ) requested to merge dz-limit-nested-groups into master
All threads resolved!

What does this MR do?

Limit level of nesting to 20

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

no

Why was this MR needed?

To prevent overuse of the feature. GitLab UI cant display infinite level of nesting anyway. The number 20 was taken based on maximum nesting level of Android repo (15) + some extra backup

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/2772

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
  • Douwe Maan
  • added 1 commit

    • f642a460 - Limit level of nesting for groups

    Compare with previous version

  • Dmytro Zaporozhets (DZ) resolved all discussions

    resolved all discussions

  • @DouweM should be better now!

  • merged

  • Douwe Maan mentioned in commit 7fa767fe

    mentioned in commit 7fa767fe

  • Dmytro Zaporozhets (DZ) added ~1492488 label

    added ~1492488 label

  • The number 20 was taken based on maximum nesting level of Android repo (15) + some extra backup

    @dzaporozhets by Android repo you mean the official Android project repository? Do you have a link for this? Thanks :bow:

  • Please register or sign in to reply
    Loading