Skip to content
Snippets Groups Projects

Exclude project namespace when computing subgroup counts

All threads resolved!

What does this MR do and why?

When loading counts for group lists exclude project namespaces from subgroup counts

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Alexandru Croitor

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
  • Brett Walker
  • Brett Walker
  • Brett Walker approved this merge request

    approved this merge request

  • @acroitor looks great! Nothing blocking, just a few questions. Going ahead an approving :rocket:

  • Brett Walker removed review request for @digitalmoksha

    removed review request for @digitalmoksha

  • :wave: @digitalmoksha, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • Alexandru Croitor mentioned in merge request !73351 (merged)

    mentioned in merge request !73351 (merged)

  • Alexandru Croitor added 351 commits

    added 351 commits

    Compare with previous version

  • Alexandru Croitor requested review from @cablett

    requested review from @cablett

  • added 1 commit

    • 9f1d766a - Exclude project namespace when computing subgroup counts

    Compare with previous version

  • charlie ablett resolved all threads

    resolved all threads

  • charlie ablett approved this merge request

    approved this merge request

  • Awesome work @acroitor :rocket: LGTM. New pipe started and MWPS set :thumbsup:

  • charlie ablett enabled an automatic merge when the pipeline for 8c257dfe succeeds

    enabled an automatic merge when the pipeline for 8c257dfe succeeds

  • charlie ablett mentioned in commit 466cf5fd

    mentioned in commit 466cf5fd

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Please register or sign in to reply
    Loading