Age | Commit message (Collapse) | Author | |
---|---|---|---|
2017-03-13 | take nonewline context into account in diff parser | mhasbini | |
2017-02-23 | Fix code for cops | Douwe Maan | |
2017-02-23 | Enable Style/ConditionalAssignment | Douwe Maan | |
2017-02-23 | Enable Performance/RedundantMatch | Douwe Maan | |
2017-02-23 | Enable Style/MutableConstant | Douwe Maan | |
2017-02-13 | Enable `Lint/EmptyWhen` cop and correct offense | Robert Speicher | |
2016-12-26 | Fix MR with files hidden by .gitattributes | Sean McGivern | |
Don't try to highlight and cache files hidden by .gitattributes entries. | |||
2016-12-02 | Fixes ActionView::Template::Error: undefined method `text?` for nil:NilClass | Valery Sizov | |
2016-11-25 | Fix: Timeout creating and viewing merge request for binary file | Valery Sizov | |
2016-11-14 | Fix Error 500 when creating a merge request that contains an image that was ↵ | Stan Hu | |
deleted and added Steps to reproduce: 1. Start with a repo with an image 2. Add a commit to delete the image 3. Add another commit to replace the image with another image In a diff comparison, we really just compare about what the image was before the diff, not the direct parent of the last commit. This MR fixes that. Closes #3893, gitlab-org/gitlab-ee#678 Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2016-11-10 | Fix expanding a collapsed diff when converting a symlink to a regular file | Adam Niedzielski | |
In this case comparing old_path and new_path is not enough because there are two entires that match. | |||
2016-10-17 | Fix the diff in the merge request view when converting a symlink to a ↵ | Adam Niedzielski | |
regular file. In this specific case using file_path as a cache key is not enough, because there are two entries with the same path. Closes #21610. | |||
2016-08-22 | Merge branch 'master' into dz-merge-request-version | Dmitriy Zaporozhets | |
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | |||
2016-08-19 | Fix line commenting for the initial commit | Sean McGivern | |
The initial commit doesn't have a parent, so explicitly pass the blank SHA and handle that when calculating the position. | |||
2016-08-19 | Merge branch 'master' into dz-merge-request-version | Dmitriy Zaporozhets | |
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | |||
2016-08-15 | Make merge request diff works with new FileCollection logic | Dmitriy Zaporozhets | |
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | |||
2016-08-12 | Fix specs | Sean McGivern | |
- Add match line header to expected result for `File#sections`. - Lowercase CSS colours. - Remove unused `diff_refs` keyword argument. - Rename `parent` -> `parent_file`, to be more explicit. - Skip an iteration when highlighting. | |||
2016-08-12 | Auto-highlight conflict when rich_text is called | Sean McGivern | |
2016-08-12 | Add backend for merge conflicts reading | Sean McGivern | |
2016-08-03 | switch from diff_file_collection to diffs | Paco Guzman | |
So we have raw_diffs too | |||
2016-08-03 | Introduce Compare model in the codebase. | Paco Guzman | |
This object will manage Gitlab::Git::Compare instances | |||
2016-08-03 | Move to Gitlab::Diff::FileCollection | Paco Guzman | |
Instead calling diff_collection.count use diff_collection.size which is cache on the diff_collection | |||
2016-08-03 | Cache highlighted diff lines for merge requests | Paco Guzman | |
Introducing the concept of SafeDiffs which relates diffs with UI highlighting. | |||
2016-07-29 | Merge branch 'rubocop/enable-access-modifiers-cops' into 'master' | Robert Speicher | |
Enable Rubocop cops that check access modifiers ## What does this MR do? This MR enables Rubocop cops that detect methods that should be restricted but are the part of public API because of access modifiers used improperly. This also fixes existing offenses. ## Why was this MR needed? Some method in our codebase are public instead of being private because it is sometimes difficult to get it right without static analysis. ## What are the relevant issue numbers? See #17478 Closes #17372 See merge request !5014 | |||
2016-07-21 | Add Discussion model to represent MR/diff discussion | Douwe Maan | |
2016-07-20 | Ensure to_json methods take optional argument | Sean McGivern | |
2016-07-19 | Fix method visibility in inline diff class | Grzegorz Bizon | |
2016-07-18 | Collapsed diffs lines/size don't accumulate to overflow diffs. | Paco Guzman | |
2016-07-13 | Merge branch 'better-parallel-diff' into 'master' | Robert Speicher | |
Actually render old and new sections of parallel diff next to each other See merge request !5173 | |||
2016-07-13 | Merge branch 'multi-line-inline-diff' into 'master' | Douwe Maan | |
Render inline diffs for multiple changed lines following eachother Before: ![Screen_Shot_2016-07-11_at_00.08.27](/uploads/b14664211e0f5cef6e77a78eadfcbcdf/Screen_Shot_2016-07-11_at_00.08.27.png) After: ![Screen_Shot_2016-07-11_at_00.07.34](/uploads/567be631869a4867a2edf6ff7eda6369/Screen_Shot_2016-07-11_at_00.07.34.png) See merge request !5174 | |||
2016-07-13 | Actually render old and new sections of parallel diff next to each other | Douwe Maan | |
2016-07-13 | Rename constant to be more descriptive | Douwe Maan | |
2016-07-12 | Extended regexes ignore whitespace, so use \s | Douwe Maan | |
2016-07-12 | Render inline diffs for multiple changed lines following eachother | Douwe Maan | |
2016-07-08 | Collapse large diffs by default | Sean McGivern | |
When rendering a list of diff files, skip those where the diff is over 10 KB and provide an endpoint to render individually instead. | |||
2016-07-07 | Remove duplication, unused methods, and some other style things | Douwe Maan | |
2016-07-07 | Add some more code comments. | Douwe Maan | |
2016-07-07 | Display new diff notes and allow creation through the web interface | Douwe Maan | |
2016-07-07 | Add Gitlab::Git::PositionTracer | Douwe Maan | |
2016-07-07 | Add Gitlab::Diff::LineMapper | Douwe Maan | |
2016-07-07 | Add Gitlab::Git::Position | Douwe Maan | |
2016-07-07 | Extract parts of LegacyDiffNote into DiffOnNote concern and move part of ↵ | Douwe Maan | |
responsibility to other classes | |||
2016-07-07 | Style diff and blob file headers the same way | Douwe Maan | |
2016-07-07 | Add DiffFile#blob and #old_blob | Douwe Maan | |
2016-07-07 | Represent DiffRefs as proper class instead of tuple array | Douwe Maan | |
2016-07-01 | Enable Style/EmptyLines cop, remove redundant ones | Grzegorz Bizon | |
2016-06-01 | Add leading comment space cop | Felipe Artur | |
2016-05-18 | Reuse existing SCSS variables | Rémy Coutable | |
Signed-off-by: Rémy Coutable <remy@rymai.me> | |||
2016-05-18 | Create DiffFilter and change SystemNoteService#change_title to use ↵ | Adam Butler | |
Gitlab::Diff::InlineDiff | |||
2016-05-11 | Enable the StringReplacement cop. | Connor Shea | |
Also fix one use of `gsub` that would be faster as `delete`. Use `tr` instead of `gsub` when you are replacing the same number of characters. Use `delete` instead of `gsub` when you are deleting characters. |