summaryrefslogtreecommitdiff
path: root/app/services/compare_service.rb
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'osw-stop-recalculating-merge-base-on-mr-loading' into 'master'Sean McGivern2018-03-131-2/+7
| | | | | Avoid re-fetching merge-base SHA from Gitaly unnecessarily See merge request gitlab-org/gitlab-ce!17630
* Ensure MRs always use branch refs for comparisonSean McGivern2017-11-281-6/+6
| | | | | | | | | | If a merge request was created with a branch name that also matched a tag name, we'd generate a comparison to or from the tag respectively, rather than the branch. Merging would still use the branch, of course. To avoid this, ensure that when we get the branch heads, we prepend the reference prefix for branches, which will ensure that we generate the correct comparison.
* Prepare GitOperationService for move to Gitlab::GitJacob Vosmaer2017-08-311-20/+2
|
* Fix Diff::Position#diff_file for positions on straight diffsdm-diff-file-straight-diffDouwe Maan2017-06-061-3/+3
|
* Rename from base to start because base could mean merge baseLin Jen-Shin2017-01-061-6/+6
|
* Rename source to base to avoid confusion from MRLin Jen-Shin2017-01-061-6/+6
|
* Introduce Repository#with_repo_branch_commitLin Jen-Shin2017-01-051-12/+6
| | | | | | | | | We merge repository checks inside it so we don't have to check it on the call site, and we could also load the commit for the caller. This greatly reduce code duplication. Feedback: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7237#note_20572919
* Should pass branch name, not commit object!Lin Jen-Shin2016-12-101-5/+6
|
* Should pass repository rather than projectLin Jen-Shin2016-12-081-1/+2
|
* Fix tests to use the new APILin Jen-Shin2016-12-081-6/+8
|
* Introduce Repository#with_tmp_ref which we needLin Jen-Shin2016-12-081-12/+18
| | | | | commits from the other repository. We'll cleanup the tmp ref after we're done with our business.
* Refactor straight compare diff codeDmitriy Zaporozhets2016-10-121-2/+2
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Use straight diff approach when compare merge request versionsDmitriy Zaporozhets2016-10-121-3/+4
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* switch from diff_file_collection to diffs20034-safe-diffsPaco Guzman2016-08-031-1/+0
| | | | So we have raw_diffs too
* Introduce Compare model in the codebase.Paco Guzman2016-08-031-2/+5
| | | This object will manage Gitlab::Git::Compare instances
* Remove unused argument to CompareService#executeSean McGivern2016-07-081-1/+1
|
* Use Gitlab::Git::DiffCollectionsJacob Vosmaer2016-03-031-7/+5
|
* Add ignore white space option in merge request diffMinsik Yoon2015-11-131-2/+2
| | | | | | fix this issue(https://gitlab.com/gitlab-org/gitlab-ce/issues/1393). Add ignore whitespace optoin to Commits Compare view
* Fix Error 500 when comparing non-existing branchesfix-issue-2593Stan Hu2015-09-181-1/+4
| | | | Closes #2593
* Revert "Merge branch 'revert-satellites' into 'master' "Dmitriy Zaporozhets2015-08-111-20/+21
| | | | | This reverts commit 5daf44b7c86e0e2641a902b1da8b01d91fa3dbfa, reversing changes made to 2f706fbd231cabe7a76a5d17ac44285aaaf8592c.
* Revert "Merge branch 'drop-satellites'"Dmitriy Zaporozhets2015-08-111-21/+20
| | | | | | | This reverts commit 957e849f41d96fa9778fcdd06792d2f0274b29ab, reversing changes made to 6b9dbe9f5a175a8162abf296367f561bab3eea1a. Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Refactor compare and fetch logicDmitriy Zaporozhets2015-07-151-20/+13
|
* Implement merge from forks without satellitesDmitriy Zaporozhets2015-07-151-1/+18
|
* Remove satellitesDmitriy Zaporozhets2015-07-151-10/+1
|
* Comment typo.Ciro Santilli2014-08-291-1/+1
|
* Comment typo.Ciro Santilli2014-08-291-1/+1
|
* Git::Compare does not have limit param any moreDmitriy Zaporozhets2014-07-291-1/+0
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Refactor compare logic for MR. Use satellites only for forks for better ↵Dmitriy Zaporozhets2014-07-291-0/+28
performance Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>