Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add latest changes from gitlab-org/gitlab@master | GitLab Bot | 2019-09-13 | 1 | -0/+2 |
| | |||||
* | Adds the Rubocop ReturnNil cop | Andrew Newdigate | 2019-03-06 | 1 | -1/+1 |
| | | | | | This style change enforces `return if ...` instead of `return nil if ...` to save maintainers a few minor review points | ||||
* | Simplify by using Gitlab::SafeRequestStore | Michael Kozono | 2018-09-24 | 1 | -5/+1 |
| | | | | These are clear wins. | ||||
* | Enable frozen string in app/models/*.rbrepo-forks/gitlab-ce-frozen-string-enable-app-models | gfyoung | 2018-07-26 | 1 | -0/+2 |
| | | | | Partially addresses #47424. | ||||
* | Improve performance of DiffDiscussion#truncated_diff_lines and ↵ | Douwe Maan | 2017-12-22 | 1 | -5/+1 |
| | | | | DiffNote#diff_line by removing expensive diff position calculation and comparison | ||||
* | Rename ActiverecordSerialize cop | Yorick Peterse | 2017-07-06 | 1 | -1/+1 |
| | | | | | This cop has been renamed to ActiveRecordSerialize to match the way "ActiveRecord" is usually written. | ||||
* | Don't display comment on unchanged line on both sides in parallel diff | Douwe Maan | 2017-06-19 | 1 | -1/+1 |
| | |||||
* | Added Cop to blacklist the use of serializedocument-not-using-serialize | Yorick Peterse | 2017-05-31 | 1 | -1/+1 |
| | | | | | This Cop blacklists the use of ActiveRecord's "serialize" method, except for cases where we already use this. | ||||
* | Pass fallback_diff_refs to Diff::File instead of using view helpers | Douwe Maan | 2017-05-23 | 1 | -1/+1 |
| | |||||
* | Link to outdated diff in older MR version from outdated diff discussion | Douwe Maan | 2017-04-08 | 1 | -1/+2 |
| | |||||
* | Refactor resolvability checks based on type | Douwe Maan | 2017-04-06 | 1 | -0/+3 |
| | |||||
* | Address review comments | Douwe Maan | 2017-04-05 | 1 | -1/+3 |
| | |||||
* | Address review comments | Douwe Maan | 2017-04-05 | 1 | -0/+1 |
| | |||||
* | Extract commonalities between DiffDiscussion and LegacyDiffDiscussion | Douwe Maan | 2017-04-05 | 1 | -8/+0 |
| | |||||
* | Add option to start a new discussion on an MR | Douwe Maan | 2017-04-05 | 1 | -8/+2 |
| | |||||
* | Fix bug where notes weren’t shown in discussion when the code had changed ↵ | Douwe Maan | 2016-08-19 | 1 | -0/+4 |
| | | | | after creation | ||||
* | Store discussion_id on Note for faster discussion lookup. | Douwe Maan | 2016-08-17 | 1 | -6/+6 |
| | |||||
* | Merge branch 'master' into diff-line-comment-vuejs | Douwe Maan | 2016-08-03 | 1 | -2/+2 |
|\ | | | | | | | | | | | # Conflicts: # app/models/discussion.rb # db/schema.rb | ||||
| * | switch from diff_file_collection to diffs20034-safe-diffs | Paco Guzman | 2016-08-03 | 1 | -2/+2 |
| | | | | | | | | So we have raw_diffs too | ||||
* | | Merge branch 'master' into diff-line-comment-vuejs | Douwe Maan | 2016-07-28 | 1 | -1/+9 |
|\ \ | |/ | | | | | | | # Conflicts: # db/schema.rb | ||||
| * | Reduce number of queries made for merge_requests/:id/diffsfix/use-fewer-queries-for-mr-notes | Ahmad Sherif | 2016-07-28 | 1 | -1/+9 |
| | | |||||
* | | Use sha1 of discussion ID. | Douwe Maan | 2016-07-25 | 1 | -1/+1 |
|/ | |||||
* | Don't fail when a LegacyDiffNote didn't store the right diff | Douwe Maan | 2016-07-15 | 1 | -2/+2 |
| | |||||
* | Extract parts of LegacyDiffNote into DiffOnNote concern and move part of ↵ | Douwe Maan | 2016-07-06 | 1 | -58/+11 |
| | | | | responsibility to other classes | ||||
* | Restore old behavior around diff notes to outdated discussions | Douwe Maan | 2016-06-22 | 1 | -1/+1 |
| | |||||
* | Incorportate feedback | Z.J. van de Weg | 2016-06-01 | 1 | -0/+4 |
| | |||||
* | Clean up LegacyDiffNote somewhat | Douwe Maan | 2016-05-13 | 1 | -98/+73 |
| | |||||
* | Extract LegacyDiffNote out of Note | Douwe Maan | 2016-05-13 | 1 | -0/+182 |