- 10 6月, 2016 3 次提交
-
-
由 Rémy Coutable 提交于
The logic of the method was obviously inverted. Signed-off-by: NRémy Coutable <remy@rymai.me>
-
由 Rémy Coutable 提交于
Based on the feedback from reviewers. Signed-off-by: NRémy Coutable <remy@rymai.me>
-
由 Rui Anderson 提交于
Signed-off-by: NRémy Coutable <remy@rymai.me>
-
- 03 6月, 2016 2 次提交
-
-
由 Kamil Trzcinski 提交于
-
由 Kamil Trzcinski 提交于
-
- 02 6月, 2016 2 次提交
-
-
由 Kamil Trzcinski 提交于
-
由 Josh Frye 提交于
-
- 01 6月, 2016 1 次提交
-
-
由 Yorick Peterse 提交于
There are several changes to this module: 1. The use of an explicit stack in Participable#participants 2. Proc behaviour has been changed 3. Batch permissions checking == Explicit Stack Participable#participants no longer uses recursion to process "self" and all child objects, instead it uses an Array and processes objects in breadth-first order. This allows us to for example create a single Gitlab::ReferenceExtractor instance and pass this to any Procs. Re-using a ReferenceExtractor removes the need for running potentially many SQL queries every time a Proc is called on a new object. == Proc Behaviour Changed Previously a Proc in Participable was expected to return an Array of User instances. This has been changed and instead it's now expected that a Proc modifies the Gitlab::ReferenceExtractor passed to it. The return value of the Proc is ignored. == Permissions Checking The method Participable#participants uses Ability.users_that_can_read_project to check if the returned users have access to the project of "self" _without_ running multiple SQL queries for every user.
-
- 30 5月, 2016 2 次提交
-
-
由 Grzegorz Bizon 提交于
-
由 Grzegorz Bizon 提交于
-
- 20 5月, 2016 1 次提交
-
-
由 Jeroen Jacobs 提交于
-
- 14 5月, 2016 1 次提交
-
-
- 10 5月, 2016 1 次提交
-
-
由 Jeroen van Baarsen 提交于
In 8278b763 the default behaviour of annotation has changes, which was causing a lot of noise in diffs. We decided in #17382 that it is better to get rid of the whole annotate gem, and instead let people look at schema.rb for the columns in a table. Fixes: #17382
-
- 12 4月, 2016 1 次提交
-
-
由 Kamil Trzcinski 提交于
-
- 29 3月, 2016 1 次提交
-
-
由 Abhi Rao 提交于
Fixes #14692
-
- 20 3月, 2016 1 次提交
-
-
由 Zeger-Jan van de Weg 提交于
-
- 19 3月, 2016 2 次提交
-
-
由 Rémy Coutable 提交于
-
由 Rémy Coutable 提交于
We have to ensure source_sha and target_sha are not nil before calling Gitlab::Git::Commit.between.
-
- 18 3月, 2016 1 次提交
-
-
- 16 3月, 2016 2 次提交
-
-
由 Pierre de La Morinerie 提交于
`MergeRequest#source_sha` is expected to return the sha of the source branch last commit. But when a open Merge Request has no diff (e.g. all commits have already been merged to the target branch), `merge_request.source_sha` incorrectly returns `nil`. This was un-noticed before – but now that !2217 has been merged, it makes `Gitlab::Git::Commit.between` raise an "Unexpected nil argument" exception. This fixes the crash, by making sure that `source_sha` returns a correct result even when there is no diff available.
-
由 Pierre de La Morinerie 提交于
-
- 12 3月, 2016 1 次提交
-
-
由 Yorick Peterse 提交于
The OR condition for source_project_id/target_project_id leads to a query plan that performs rather poorly on PostgreSQL due to the use of sub-queries. Because Rails offers no easy alternative for this particular problem we're forced to using a UNION for both conditions. The resulting query performs much faster than just using an OR.
-
- 03 3月, 2016 1 次提交
-
-
由 Pierre de La Morinerie 提交于
-
- 20 2月, 2016 1 次提交
-
-
由 Rubén Dávila 提交于
-
- 12 2月, 2016 1 次提交
-
-
由 Kirill Zaitsev 提交于
- Add `avatar_url`, `description`, `git_ssh_url`, `git_http_url`, `path_with_namespace` and `default_branch` in `project` in push, issue, merge-request and note webhooks data - Deprecate the `ssh_url` in favor of `git_ssh_url` and `http_url` in favor of `git_http_url` in `project` for push, issue, merge-request and note webhooks data - Deprecate the `repository` key in push, issue, merge-request and note webhooks data, use `project` instead
-
- 04 2月, 2016 1 次提交
-
-
由 Zeger-Jan van de Weg 提交于
Fixes #3339 This MR hides the 'Remove source branch' button when a new commit is added to the source branch
-
- 01 2月, 2016 1 次提交
-
-
由 Yorick Peterse 提交于
Instead of running ClosingIssueExtractor for every commit in a merge request we can gather all the commit messages (and the merge request description), concatenate all this together and then run ClosingIssueExtractor only once. The result of this is that MergeRequest#closes_issues is now between 3.5x and 4x faster than the old setup. Using a merge request with 10 commits (each referencing a number of issues to close) this reduced the call duration from around 200 milliseconds to around 50 milliseconds. As a result of these changes the Jira related tests for MergeRequest#closes_issues have been removed. These tests stubbed Commit#closes_issues meaning that the only code that was really tested was the call to Array#uniq to filter out duplicate issues. As this code is no longer used (nor present) the corresponding tests were removed. Related: gitlab-org/gitlab-ce#12419
-
- 07 1月, 2016 1 次提交
-
-
由 Ted Hogan 提交于
-
- 06 1月, 2016 1 次提交
-
-
由 Stan Hu 提交于
-
- 19 12月, 2015 1 次提交
-
-
由 Drew Blessing 提交于
-
- 09 12月, 2015 1 次提交
-
-
由 Douwe Maan 提交于
-
- 07 12月, 2015 1 次提交
-
-
由 Zeger-Jan van de Weg 提交于
-
- 05 12月, 2015 1 次提交
-
-
由 Zeger-Jan van de Weg 提交于
-
- 02 12月, 2015 1 次提交
-
-
由 Zeger-Jan van de Weg 提交于
-
- 24 11月, 2015 1 次提交
-
-
由 Ted Hogan 提交于
Modified changelog
-
- 21 11月, 2015 1 次提交
-
-
由 Yorick Peterse 提交于
When calling MergeRequest#ci_commit the code would previously raise an error if the source project no longer existed (e.g. because the user removed their fork). See #3599 for more information.
-
- 18 11月, 2015 1 次提交
-
-
由 Zeger-Jan van de Weg 提交于
-
- 14 11月, 2015 1 次提交
-
-
由 Dmitriy Zaporozhets 提交于
Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
- 23 10月, 2015 1 次提交
-
-
由 Valery Sizov 提交于
-
- 09 10月, 2015 1 次提交
-
-
由 Ben Boeckel 提交于
-