Skip to content
Snippets Groups Projects

For-Each-Ref RPC&Messages

Merged username-removed-487110 requested to merge 66-ci-status-environment into master
All threads resolved!

Conversation: gitaly#221

Closes gitaly#81 (closed)

I ordered everything by type (service, message) to make it easier to find stuff later on when this file explodes in size.

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
  • username-removed-487110 unmarked as a Work In Progress

    unmarked as a Work In Progress

  • assigned to @bkc

  • added 1 commit

    Compare with previous version

  • Andrew Newdigate mentioned in merge request gitaly!73 (merged)

    mentioned in merge request gitaly!73 (merged)

  • assigned to @bkc

  • @bkc could you update the title and description of this MR? They will become a Git commit message when we merge. Something like I think sounds strange. And ci_status && ci_environment_status is too cryptic a commit subject line.

  • username-removed-487110 changed title from ci_status && ci_environment_status to For-Each-Ref RPC&Messages

    changed title from ci_status && ci_environment_status to For-Each-Ref RPC&Messages

  • Assigned to @eReGeBe to have a third set of eyes :) and /cc @ahmadsherif to see if he's pleased with the changes he mentioned :slight_smile:

  • added 1 commit

    • 7b3b7f37 - name => names because array

    Compare with previous version

  • @bkc left a couple of comments :thumbsup:

  • assigned to @bkc

  • added 2 commits

    • 2e74a702 - No more CommitObject for IsAncestor
    • db19a166 - No more CommitObject at all (really this time)

    Compare with previous version

  • added 1 commit

    • c2752d79 - first-class response for for-each-ref

    Compare with previous version

  • username-removed-367626 mentioned in merge request !2 (merged)

    mentioned in merge request !2 (merged)

  • added 1 commit

    Compare with previous version

  • @jacobvosmaer-gitlab Fixed the things you commented on.

  • added 12 commits

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Rebased and fixed all the merge conflicts

  • Jacob Vosmaer (GitLab) resolved all discussions

    resolved all discussions

  • Jacob Vosmaer (GitLab) enabled an automatic merge when the pipeline for e13050f6 succeeds

    enabled an automatic merge when the pipeline for e13050f6 succeeds

  • mentioned in commit 1d4fb881

  • mentioned in issue gitaly#221

  • mentioned in issue gitaly#308

  • Please register or sign in to reply
    Loading