- 30 Oct, 2018 35 commits
-
-
gfyoung authored
Enables frozen string for all remaining files in lib/gitlab/ci. Partially addresses #47424.
-
Achilleas Pipinellis authored
Fix structure and add notification level See merge request gitlab-org/gitlab-ce!22528
-
Evan Read authored
-
Stan Hu authored
Enable some frozen string for lib/gitlab/ci See merge request gitlab-org/gitlab-ce!22636
-
Clement Ho authored
Fix application layout form description See merge request gitlab-org/gitlab-ce!22658
-
Andreas Kämmerle authored
-
Mike Greiling authored
Resolve "Create new group: Rename form fields and update UI" Closes #50962 See merge request gitlab-org/gitlab-ce!22418
-
Martin Wortschack authored
-
Clement Ho authored
Fix search "all in GitLab" not working with relative URLs Closes #53259 See merge request gitlab-org/gitlab-ce!22644
-
Mike Greiling authored
Adds missing i18n to pipelines table See merge request gitlab-org/gitlab-ce!22690
-
Phil Hughes authored
Resolve "Empty list for links to changed pages in Review App from MR Widget" Closes #53227 See merge request gitlab-org/gitlab-ce!22665
-
Filipa Lacerda authored
-
Douglas Barbosa Alexandre authored
Enhance performance of counting local LFS objects Closes gitlab-ee#6067 See merge request gitlab-org/gitlab-ce!22143
-
Filipa Lacerda authored
Add dynamic timer for delayed jobs in pipelines list See merge request gitlab-org/gitlab-ce!22621
-
Filipa Lacerda authored
Backport ee 7203 sticky logs topbar See merge request gitlab-org/gitlab-ce!22282
-
Simon Knox authored
-
Filipa Lacerda authored
Fixes diff discussions not being fully removed Closes #53317 See merge request gitlab-org/gitlab-ce!22680
-
Sean McGivern authored
Fix extra merge request versions created from forked merge requests Closes #53153 See merge request gitlab-org/gitlab-ce!22611
-
Sean McGivern authored
Wait for push before checking for new commit Closes gitlab-org/quality/staging#14 See merge request gitlab-org/gitlab-ce!22633
-
Filipa Lacerda authored
-
Filipa Lacerda authored
When a deployment has no files to show in the dropdown we fallback to the regular single button for the review app link. This commit moves the review app link into a component since it's used twice.
-
Phil Hughes authored
This fixes a bug where a discussion on a none changed line would not get fully removed and therefore leave the comment row empty. This was caused by the discussiob being added to the right when it shouldnt of been This also fixes a very rare edge case where discussions would get added twice to diff lines causing a Vue rendering warning Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53317
-
Dmitriy Zaporozhets authored
Fix #53298 JupyterHub restarts should work without errors Closes #53298 See merge request gitlab-org/gitlab-ce!22671
-
Marin Jankovski authored
Fix Review Apps testing guide See merge request gitlab-org/gitlab-ce!22670
-
Grzegorz Bizon authored
Renaming Secret Variables in the codebase Closes #49629 See merge request gitlab-org/gitlab-ce!22414
-
Marcel Amirault authored
-
Grzegorz Bizon authored
Adding tests for filtering activities/comments in issues Closes #51889 See merge request gitlab-org/gitlab-ce!22564
-
Ramya Authappan authored
-
Toon Claes authored
Add an index to the `file_store` column on `lfs_objects`. This makes counting local objects faster. Also, there is no longer need to check for objects with `file_store` being `NULL`. See https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/18557 --- ### Query plans #### Before & with `NULL` ``` Aggregate (cost=113495.96..113495.97 rows=1 width=8) (actual time=1691.394..1691.394 rows=1 loops=1) -> Seq Scan on lfs_objects (cost=0.00..106415.50 rows=2832186 width=0) (actual time=0.012..1312.488 rows=2852607 loops=1) Filter: ((file_store = 1) OR (file_store IS NULL)) Rows Removed by Filter: 131 Planning time: 0.077 ms Execution time: 1691.433 ms ``` #### Before, without `NULL` ``` Aggregate (cost=113495.96..113495.97 rows=1 width=8) (actual time=856.423..856.424 rows=1 loops=1) -> Seq Scan on lfs_objects (cost=0.00..106415.50 rows=2832186 width=0) (actual time=0.012..672.181 rows=2852607 loops=1) Filter: (file_store = 1) Rows Removed by Filter: 131 Planning time: 0.128 ms Execution time: 856.470 ms ``` #### After & with `NULL` ``` Aggregate (cost=68819.95..68819.96 rows=1 width=8) (actual time=583.355..583.355 rows=1 loops=1) -> Index Only Scan using index_lfs_objects_on_file_store on lfs_objects (cost=0.43..61688.35 rows=2852643 width=0) (actual time=0.028..399.177 rows=2852607 loops=1) Filter: ((file_store = 1) OR (file_store IS NULL)) Rows Removed by Filter: 131 Heap Fetches: 867 Planning time: 0.096 ms Execution time: 583.404 ms ``` #### After, without `NULL` ``` Aggregate (cost=68817.29..68817.30 rows=1 width=8) (actual time=490.550..490.551 rows=1 loops=1) -> Index Only Scan using index_lfs_objects_on_file_store on lfs_objects (cost=0.43..61685.68 rows=2852643 width=0) (actual time=0.040..311.760 rows=2852607 loops=1) Index Cond: (file_store = 1) Heap Fetches: 831 Planning time: 0.294 ms Execution time: 490.590 ms ``` Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/6067
-
Jan Provaznik authored
-
Stan Hu authored
Fix problem with spec that showed up in EE merge See merge request gitlab-org/gitlab-ce!22675
-
Evan Read authored
docs: Add deprecation notice for renamed licensed feature See merge request gitlab-org/gitlab-ce!22575
-
Brett Walker authored
-
Evan Read authored
Added a note about machine types See merge request gitlab-org/gitlab-ce!22572
-
Mark Veenstra authored
-
- 29 Oct, 2018 5 commits
-
-
Stan Hu authored
When a forked merge request was created with the same branch name as the target name, MergeRequests::RefreshService would always create a new diff even though nothing had changed. For example, on GitLab.com: 1. There were a number of merge requests in the gitlab-ce and www-gitlab-com projects that had old merge requests from the community. 2. These merge requests originated from forked projects and used the source branch master. 3. When someone pushed to master in the main repository, MergeRequests::RefreshService would see that master matched the merge requests in question and generated a new diff. 4. This led to an explosion of merge request diffs and slowed down the "Changes" tab considerably. This change alters MergeRequests::RefreshService so that it will only refresh the diff if the merge request's source project and branch match. Otherwise, the refresh will only happen if a pushed commit contains a commit relevant to the existing merge request. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/53153
-
Mike Greiling authored
Update gitlab-ui to 1.9.0 See merge request gitlab-org/gitlab-ce!22632
-
Clement Ho authored
-
Mike Greiling authored
Remove mousetrap-rails gem Closes #53270 See merge request gitlab-org/gitlab-ce!22647
-
DJ Mountney authored
-