Skip to content
Snippets Groups Projects
  1. Mar 08, 2016
  2. Mar 07, 2016
    • Douwe Maan's avatar
      Merge branch 'issue_13621_2' into 'master' · b8f6e084
      Douwe Maan authored
      Labels should be visible in dashboard and group milestone views
      
      Closes #13621
      
      See merge request !2931
      
      Former-commit-id: 903aa7c9
      b8f6e084
    • Douwe Maan's avatar
      Merge branch 'feature/cross-project-labels' into 'master' · b67e42fc
      Douwe Maan authored
      Add support for cross project references for labels
      
      ## Summary
      
      Support for cross project references for labels.
      
      ## Rationale
      
      1.   Cross project label references are currently not supported in GitLab
      1.   `to_reference` method signature in `Label` model breaks the abstraction introduced in `Referable`.
      
            `concerns/referable.rb:  def to_reference(_from_project = nil)`
      
            Signatures:
      
            ```
            label.rb:           def to_reference(format = :id)
      
            commit_range.rb:    def to_reference(from_project = nil)
            commit.rb:          def to_reference(from_project = nil)
            external_issue.rb:  def to_reference(_from_project = nil)
            group.rb:           def to_reference(_from_project = nil)
            issue.rb:           def to_reference(from_project = nil)
            merge_request.rb:   def to_reference(from_project = nil)
            milestone.rb:       def to_reference(from_project = nil)
            project.rb:         def to_reference(_from_project = nil)
            snippet.rb:         def to_reference(from_project = nil)
            user.rb:            def to_reference(_from_project = nil)
            ```
      
           This MR suggests using `def to_reference(from_project = nil, format: :id)` which makes use of keyword arguments and preserves abstract interface.
      
      1.   We need support for cross project label references when we want to move issue to another project
      
           It may happen that issue description, system notes or comments contain reference to label and this reference will be invalid after moving issue to another project and will not be displayed correctly unless we have support for cross project references.
      
           Merge request that needs this feature: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/2831
      
      
      I think that cross project label references may be useful, (example: `Hey, see our issues for CI in GitLab CE! - gitab-org/gitlab-ce~"CI"`).
      
      cc @JobV @DouweM @rspeicher 
      
      See merge request !2966
      
      Former-commit-id: 99f08b3f
      b67e42fc
    • Douwe Maan's avatar
      Merge branch 'rs-factory-nitpicks' into 'master' · 639c8610
      Douwe Maan authored
      More Factory cleanup
      
      Addresses nitpicks from https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/2847
      
      See merge request !3108
      
      Former-commit-id: eab7892d
      639c8610
    • Douwe Maan's avatar
      Merge branch 'indicate-mr-diverged-from-target' into 'master' · 51f476e0
      Douwe Maan authored
      Indicate when an MR diverged from the target branch
      
      This adds an indicator to the "Merge MR" box, to tell if and how much an MR diverged from its target branch.
      
      For instance, consider an MR to merge the branch `feature` into `master`. Some other commits were added to `master` since `feature` was created, and the two branches diverged.
      
      ```text
      o master
      |
      o    o feature
      |    |
      o    o
      |  /
      o
      ```
      
      In this case, there will be a label in the MR Merge box stating:
      
      > This MR is by 3 commits behind the target branch `master`.
      
      ## Screenshots
      
      ### The branch diverged from the target (UI Proposal)
      
      ![UI_suggestion_1](/uploads/cd5bee3959e68026ec7d5097259d53f4/UI_suggestion_1.png)
      
      ### The branch diverged from the target (alternative UI Proposal)
      
      ![UI_suggestion_2](/uploads/f36977101b59a610850e129837dfbc83/UI_suggestion_2.png)
      
      ## How is this useful?
      
      - In a _rebase-workflow_ (MR are preferably rebased before being merged), the reviewer wants to know if an MR is rebased on the target branch before merging it. 
          
          _With this indicator, the reviewer knows immediately if the branch is rebased, or if she needs to ask the committer to rebase its branch._
      
      <br>
      
      - To keep the git history readable, a team prefers to avoid merging branches that really lag a lot behind the target branch. Merging an MR that is 10 commits behind is fine, but 200 is too much.
      
          _With this indicator, the reviewer can see on the MR page if the branch is really far behind the target – or only a few commits behind._
      
      ## Open questions
      
      We've been using this at @captaintrain for a few months now, and found it quite useful.
      
      I guess the open-questions are mostly: what UI would be the more adequate? Any thoughts on this, on the general usefulness and/or on the code?
      
      See merge request !2217
      
      Former-commit-id: d43c7784
      51f476e0
    • Douwe Maan's avatar
      Merge branch 'rs-note-delegates-mentionable' · 295bfff1
      Douwe Maan authored
      # Conflicts:
      #	app/models/note.rb
      
      
      Former-commit-id: be1ae2d6
      295bfff1
    • Douwe Maan's avatar
      Merge branch 'rs-remove-diffline' into 'master' · b792c3ce
      Douwe Maan authored
      Remove unused DiffLine model
      
      Added in Mar 2014. Only usage removed in Sep 2014.
      
      See merge request !3105
      
      Former-commit-id: 49d7a293
      b792c3ce
    • Douwe Maan's avatar
      Merge branch 'rs-note-reuse-for-methods' into 'master' · 3d7f88c1
      Douwe Maan authored
      Make better use of the `Note#for_*?` methods
      
      We've got all these polymorphic type-checking methods, might as well use
      'em everywhere it makes sense.
      
      See merge request !3106
      
      Former-commit-id: de872f26
      3d7f88c1
    • Douwe Maan's avatar
      Merge branch 'rs-note-model-cleanup' into 'master' · 4ae62d33
      Douwe Maan authored
      Minor Note model cleanup
      
      Remove `Note#noteable_type_name`
      
      This method was added in Dec 2012, its single use was removed two days
      later, and it's been there ever since.
      
      ---
      
      Remove `Note#system?` "override"
      
      This method was duplicating default Rails behavior.
      
      See merge request !3104
      
      Former-commit-id: 20ed08ed
      4ae62d33
    • Rubén Dávila's avatar
      Updates from last code review. · 08e076ba
      Rubén Dávila authored
      Former-commit-id: 95b06a62
      08e076ba
  3. Mar 06, 2016
  4. Mar 05, 2016
Loading