- 28 Nov, 2017 5 commits
-
-
Sean McGivern authored
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.
-
Sean McGivern authored
The st_commits and st_diffs columns on merge_request_diffs historically held the YAML-serialised data for a merge request diff, in a variety of formats. Since 9.5, these have been migrated in the background to two new tables: merge_request_diff_commits and merge_request_diff_files. That has the advantage that we can actually query the data (for instance, to find out how many commits we've stored), and that it can't be in a variety of formats, but must match the new schema. This is the final step of that journey, where we drop those columns and remove all references to them. This is a breaking change to the importer, because we can no longer import diffs created in the old format, and we cannot guarantee the export will be in the new format unless it was generated after this commit.
-
Shinya Maeda authored
- 27 Nov, 2017 1 commit
-
-
haseeb authored
-
- 25 Nov, 2017 1 commit
-
-
Vitaliy @blackst0ne Klachkov authored
-
- 24 Nov, 2017 1 commit
-
-
James Edwards-Jones authored
Previously these were duplicated so they could be different for push/merge, but this was no longer necessary after https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/11232
-
- 23 Nov, 2017 1 commit
-
-
Filipa Lacerda authored
-
- 21 Nov, 2017 1 commit
-
-
Zeger-Jan van de Weg authored
After installing a new gem, batch-loader, a construct can be used to queue data to be fetched in bulk. The gem was also introduced in both gitlab-org/gitlab-ce!14680 and gitlab-org/gitlab-ce!14846, but those mrs are not merged yet. For the generation of diffs, both the old blob and the new blob need to be loaded. This for every file in the diff, too. Now we collect all these so we do 1 fetch. Three `.allow_n_plus_1_calls` have been removed, which I expect to be valid, but this needs to be confirmed by a full CI run. Possibly closes: - https://gitlab.com/gitlab-org/gitlab-ce/issues/37445 - https://gitlab.com/gitlab-org/gitlab-ce/issues/37599 - https://gitlab.com/gitlab-org/gitlab-ce/issues/37431
-
- 17 Nov, 2017 1 commit
-
-
Francisco Javier López authored
-
- 16 Nov, 2017 2 commits
-
-
Jacopo authored
Adds a rubocop rule (with autocorrect) to ensure line break after guard clauses.
-
Yorick Peterse authored
This adds an optimised way of getting the latest pipeline status for a list of Commit objects (or just a single one).
-
- 08 Nov, 2017 1 commit
-
-
Douwe Maan authored
Use Commit#notes and Note.for_commit_id when possible to make sure we use all the indexes available to us
-
- 07 Nov, 2017 1 commit
-
-
Jarka Kadlecova authored
-
- 06 Nov, 2017 7 commits
-
-
Shinya Maeda authored
-
Alessio Caiazza authored
-
Shinya Maeda authored
-
Phil Hughes authored
updated controller based on review
-
Phil Hughes authored
-
Phil Hughes authored
also merges data in after a fetch to save overriting all data. changed the render keys to save a full re-render when any data changes
-
Phil Hughes authored
Closes #38360
-
- 03 Nov, 2017 4 commits
-
-
Alessio Caiazza authored
-
Alessio Caiazza authored
-
micael.bergeron authored
also, I refactored the MergeRequest#fetch_ref method to express the side-effect that this method has. MergeRequest#fetch_ref -> MergeRequest#fetch_ref! Repository#fetch_source_branch -> Repository#fetch_source_branch!
-
Alessio Caiazza authored
-
- 02 Nov, 2017 6 commits
-
-
Shinya Maeda authored
-
Kamil Trzcinski authored
-
Shinya Maeda authored
Fix out of sync with KubernetesService. Remove namespace pramas from controller. Use time_with_zone in schema. Remove Gcp::Clusters from safe_model_attributes.ym
-
Kamil Trzcinski authored
-
Douwe Maan authored
-
Jarka Kadlecova authored
-
- 01 Nov, 2017 4 commits
-
-
Shinya Maeda authored
-
Francisco Javier López authored
-
Kamil Trzcinski authored
-
Kamil Trzcinski authored
-
- 31 Oct, 2017 4 commits
-
-
Shinya Maeda authored
-
Eric Eastwood authored
-
Felipe Artur authored
-
Shinya Maeda authored
-