summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Instrument all service classesinstrument-service-classesYorick Peterse2016-04-121-0/+12
| | | | Fixes gitlab-org/gitlab-ce#15162
* Merge branch 'api-labels-info' into 'master' Rémy Coutable2016-04-123-20/+22
|\ | | | | | | | | | | | | API: Expose open_issues_count, closed_issues_count, open_merge_requests_count on labels Partly fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/13238 See merge request !3569
| * Add changelog itemRobert Schilling2016-04-061-0/+1
| |
| * API: Expose open_issues_count, closed_issues_count, ↵Robert Schilling2016-04-062-20/+21
| | | | | | | | open_merge_requests_count on labels
* | Merge branch 'issue_1752' into 'master' Robert Speicher2016-04-112-4/+20
|\ \ | | | | | | | | | | | | | | | | | | Fix milestone removal problem when editing issues fixes #1752 See merge request !3571
| * | improve specs code syntaxissue_1752Felipe Artur2016-04-071-11/+11
| | |
| * | Fix milestone removal problem when editing issuesFelipe Artur2016-04-062-0/+16
| | |
* | | Merge branch 'banzai-metrics' into 'master' Robert Speicher2016-04-112-8/+35
|\ \ \ | | | | | | | | | | | | | | | | Instrument Banzai code See merge request !3659
| * | | Instrument Banzai codeYorick Peterse2016-04-112-8/+35
| | | |
* | | | Merge branch 'fix-revert-scroll' into 'master' Jacob Schatz2016-04-111-0/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix scroll bars shown in Revert dialogue box ## What does this MR do? Eliminate scroll bars shown in Revert dialogue box, see the screenshots. ## Are there points in the code the reviewer needs to double check? N/A ## Why was this MR needed? Ugly scroll bars are shown in Revert dialogue box. ## What are the relevant issue numbers? N/A ## Screenshots ![Screen_Shot_2016-04-05_at_15.27.10](/uploads/428295858ada7cd2fcdf1e831c42f8ec/Screen_Shot_2016-04-05_at_15.27.10.png) See merge request !3532
| * | | | Eliminate ugly scroll bars shown in modal boxesP.S.V.R2016-04-051-0/+3
| | | | |
* | | | | Merge branch 'workhorse-0.7.1' into 'master' Yorick Peterse2016-04-113-3/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Go back to gitlab-workhorse 0.7.1 See merge request !3656
| * | | | | Go back to gitlab-workhorse 0.7.1Jacob Vosmaer2016-04-113-3/+3
|/ / / / / | | | | | | | | | | | | | | | | | | | | We found out that 0.7.2 breaks shallow `git clone`. GitLab 8.7 will work fine with gitlab-workhorse 0.7.1 too.
* | | | | Merge branch 'dashboard-filter-milestone' into 'master' Jacob Schatz2016-04-112-1/+61
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed issue with dashboard/issues not filtering by milestone Closes #15128 See merge request !3650
| * | | | | Fixed issue with dashboard/issues not filtering by milestonedashboard-filter-milestonePhil Hughes2016-04-112-1/+61
| | | | | | | | | | | | | | | | | | | | | | | | Closes #15128
* | | | | | Merge branch 'complexity/descrease-abc-size-threshold' into 'master' Dmitriy Zaporozhets2016-04-111-1/+1
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Decrease threshold for ABC Size metric in Rubocop To 55. References #3444 See merge request !3598
| * | | | | Decrease threshold for ABC Size metric in Rubocopcomplexity/descrease-abc-size-thresholdGrzegorz Bizon2016-04-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | To 60.
* | | | | | Merge branch 'metrics-measure-block-transaction' into 'master' Yorick Peterse2016-04-115-40/+67
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Store block timings as transaction values See merge request !3649
| * | | | | | Fixed stubbing for Gitlab::Metrics specsYorick Peterse2016-04-112-5/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the measure method uses Transaction.current directly the SQL subscriber (Subscribers::ActiveRecord) will add timings of queries triggered by DB cleaner.
| * | | | | | Track call counts in Gitlab::Metrics.measure_blockmetrics-measure-block-transactionYorick Peterse2016-04-113-3/+8
| | | | | | |
| * | | | | | Added specs for Gitlab::Metrics::System.cpu_timeYorick Peterse2016-04-111-0/+6
| | | | | | |
| * | | | | | Store block timings as transaction valuesYorick Peterse2016-04-114-39/+47
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes it easier to query, simplifies the code, and makes it possible to figure out what transaction the data belongs to (simply because it's now stored _in_ the transaction). This new setup keeps track of both the real/wall time _and_ CPU time spent in a block, both measured using milliseconds (to keep all units the same).
* | | | | | Merge branch 'rs-form_errors' into 'master' Dmitriy Zaporozhets2016-04-1133-153/+105
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Standardize the way we check for and display form errors - Some views had a "Close" button. We've removed this, because we don't want users accidentally hiding the validation errors and not knowing what needs to be fixed. - Some views used `li`, some used `p`, some used `span`. We've standardized on `li`. - Some views only showed the first error. We've standardized on showing all of them. - Some views added an `#error_explanation` div, which we've made standard. See merge request !3531
| * | | | | | Standardize the way we check for and display form errorsrs-form_errorsRobert Speicher2016-04-0433-153/+105
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Some views had a "Close" button. We've removed this, because we don't want users accidentally hiding the validation errors and not knowing what needs to be fixed. - Some views used `li`, some used `p`, some used `span`. We've standardized on `li`. - Some views only showed the first error. We've standardized on showing all of them. - Some views added an `#error_explanation` div, which we've made standard.
* | | | | | | Merge branch 'rs-code-review-guide' into 'master' Dmitriy Zaporozhets2016-04-112-0/+80
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First pass at a Code Review guide Largely borrowed from thoughtbot's code review guide, so attribution is included. [ci skip] See merge request !3639
| * | | | | | | Re-add a note about sarcasm to the Code Review guiders-code-review-guideRobert Speicher2016-04-101-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
| * | | | | | | First pass at a Code Review guideRobert Speicher2016-04-092-0/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Largely borrowed from thoughtbot's code review guide, so attribution is included. [ci skip]
* | | | | | | | Merge branch 'mr-ui-update' into 'master' Jacob Schatz2016-04-1122-115/+297
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated UI for new merge request Closes #2540 ![Screen_Shot_2016-03-29_at_12.53.18](/uploads/426e603675171f0dc4e0af83c7762eba/Screen_Shot_2016-03-29_at_12.53.18.png) ![Screen_Shot_2016-03-29_at_12.53.13](/uploads/645ad7955acfb9f7693245fcc048ee49/Screen_Shot_2016-03-29_at_12.53.13.png) See merge request !3228
| * | | | | | | | Merge request mobile spacingmr-ui-updatePhil Hughes2016-04-111-2/+11
| | | | | | | | |
| * | | | | | | | Fixed dropdown title overlapPhil Hughes2016-04-111-2/+2
| | | | | | | | |
| * | | | | | | | Addressed feedbackPhil Hughes2016-04-115-16/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed important from css
| * | | | | | | | Fixed testsPhil Hughes2016-04-071-1/+1
| | | | | | | | |
| * | | | | | | | Build fixPhil Hughes2016-04-062-3/+12
| | | | | | | | |
| * | | | | | | | Fixed buildsPhil Hughes2016-04-067-23/+30
| | | | | | | | |
| * | | | | | | | Fixed spacing in MR footerPhil Hughes2016-04-061-1/+1
| | | | | | | | |
| * | | | | | | | Removed un-used get requestPhil Hughes2016-04-061-2/+0
| | | | | | | | |
| * | | | | | | | Fixed target project updatePhil Hughes2016-04-066-38/+42
| | | | | | | | |
| * | | | | | | | Filters by text on elementPhil Hughes2016-04-062-7/+31
| | | | | | | | |
| * | | | | | | | Checkmarks in dropdowns for already selected valuesPhil Hughes2016-04-062-7/+8
| | | | | | | | |
| * | | | | | | | Use new dropdowns for MR comparePhil Hughes2016-04-065-32/+82
| | | | | | | | |
| * | | | | | | | Fixed failing testsPhil Hughes2016-04-061-2/+2
| | | | | | | | |
| * | | | | | | | Updated UI for new merge requestPhil Hughes2016-04-0610-61/+138
| | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | Closes #2540
* | | | | | | | Merge branch 'rs-remove-markdown-tips' into 'master' Dmitriy Zaporozhets2016-04-112-30/+0
|\ \ \ \ \ \ \ \ | |_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove `random_markdown_tip` helper Only usage got removed in !3442 See merge request !3637
| * | | | | | | Remove `random_markdown_tip` helperrs-remove-markdown-tipsRobert Speicher2016-04-092-30/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only usage got removed in !3442
* | | | | | | | Merge branch 'patch-1' into 'master' Achilleas Pipinellis2016-04-111-7/+6
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Correct documentation on enabling artifacts. The docs say to enable artifacts that settings should be set to "false". This updates the documents to change the setting to "true". See merge request !3630
| * | | | | | | | fix typoBenjamin Montgomery2016-04-101-1/+3
| | | | | | | | |
| * | | | | | | | Make the purpose of artifact config options clearerBenjamin Montgomery2016-04-101-9/+6
| | | | | | | | |
| * | | | | | | | actually enable artifactsBenjamin Montgomery2016-04-091-2/+2
| | | | | | | | |
* | | | | | | | | Merge branch 'add-missing-mixin' into 'master' Stan Hu2016-04-111-4/+6
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use false/true mixin in migration from !3640 See merge request !3646
| * | | | | | | | | Use false/true mixin in migration from !3640Stan Hu2016-04-101-4/+6
|/ / / / / / / / /