summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'build-statuses' into 'master' Rémy Coutable2016-12-1458-170/+1125
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Detailed build statuses ## What does this MR do? Implements detailed statuses for `Ci::Builds` and `CommitStatus`. It also adds a new icon for manual build. ## Screenshots ![manual_builds_icon](/uploads/22b5c594350856c85398ef705a635f8b/manual_builds_icon.png) ## What are the relevant issue numbers? See #24273, closes #22642 See merge request !7989
| * Improve build status specs contexts descriptionsGrzegorz Bizon2016-12-145-4/+6
| |
| * Add some missing tests for detailed status methodsbuild-statusesGrzegorz Bizon2016-12-143-2/+32
| |
| * Remove trailing blank line from Allowable moduleGrzegorz Bizon2016-12-141-1/+0
| |
| * Update manual build icon SVGGrzegorz Bizon2016-12-131-1/+1
| |
| * Make it possible to mix `Gitlab::Routing` inGrzegorz Bizon2016-12-133-1/+30
| |
| * Extract abilities checking module from ability modelGrzegorz Bizon2016-12-134-7/+36
| |
| * Extend tests for pipeline detailed status helpersGrzegorz Bizon2016-12-131-12/+18
| |
| * Add tests for common build detailed status helpersGrzegorz Bizon2016-12-131-0/+37
| |
| * Add missing tests for build `cancelable?` methodGrzegorz Bizon2016-12-131-0/+36
| |
| * Add tests for detailed build statuses factoryGrzegorz Bizon2016-12-131-0/+141
| |
| * Make it possible to retry build that was canceledGrzegorz Bizon2016-12-132-1/+9
| |
| * Make build retryable only if complete and executedGrzegorz Bizon2016-12-132-9/+32
| |
| * Improve readability in methods for detailed statusGrzegorz Bizon2016-12-134-4/+12
| |
| * Add tests for build cancelable/retryable statusesGrzegorz Bizon2016-12-132-0/+172
| |
| * Extend specs for build play/stop detailed statusesGrzegorz Bizon2016-12-132-7/+55
| |
| * Refine build stop/play extended status specsGrzegorz Bizon2016-12-133-35/+35
| |
| * Use manual build icon in play/stop build statusesGrzegorz Bizon2016-12-133-6/+12
| |
| * Adds manual action icon and case to show itFilipa Lacerda2016-12-132-0/+3
| |
| * Fix detailed status specs for pipeline stage modelGrzegorz Bizon2016-12-121-1/+3
| |
| * Fix detailed status badge for generic commit statusGrzegorz Bizon2016-12-121-1/+1
| | | | | | | | [ci skip]
| * Fix pipeline specs for detailed statusesGrzegorz Bizon2016-12-121-8/+12
| | | | | | | | [ci skip]
| * Add specs for build stop extended detailed statusGrzegorz Bizon2016-12-122-4/+59
| |
| * Add tests for build play extended detailed statusGrzegorz Bizon2016-12-122-3/+69
| |
| * Fix build stop extended status CSS classGrzegorz Bizon2016-12-121-4/+0
| |
| * Fix path to build status details in common helpersGrzegorz Bizon2016-12-121-1/+1
| |
| * Improve detailed status badge partialGrzegorz Bizon2016-12-128-18/+14
| |
| * Revert some unneeded changesKamil Trzcinski2016-12-121-1/+1
| |
| * Add action_class/action_titleKamil Trzcinski2016-12-125-3/+30
| |
| * Rename icon_with_label to icon_with_descriptionKamil Trzcinski2016-12-128-8/+9
| |
| * Fix auto loading of constants for Ci StatusesKamil Trzcinski2016-12-129-20/+21
| |
| * Fix some detailed statuses specs related to abilitiesGrzegorz Bizon2016-12-1210-18/+49
| |
| * Fix tests related to detailed statuses and permissionsGrzegorz Bizon2016-12-1214-41/+45
| | | | | | | | [ci skip]
| * Incorporate permission checks into new CI statusesGrzegorz Bizon2016-12-1211-49/+60
| | | | | | | | [ci skip]
| * Refactor ci status factories to DRY code a littleGrzegorz Bizon2016-12-125-29/+28
| |
| * Remove ci_status_with_icon helper and replace it with partial [ci skip]Kamil Trzcinski2016-12-128-34/+17
| |
| * Check permission of detailsKamil Trzcinski2016-12-124-7/+7
| |
| * Introduce `cancelable` and `returnable` [ci skip]Kamil Trzcinski2016-12-125-35/+64
| |
| * Improve actionsKamil Trzcinski2016-12-125-18/+110
| |
| * Added Ci::Status::BuildKamil Trzcinski2016-12-125-3/+84
| |
* | Merge branch '25651-fix-master' into 'master' Rémy Coutable2016-12-141-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix wrong error message expectation in API::Commits spec See commit message or https://gitlab.com/gitlab-org/gitlab-ce/issues/25651#note_19996076 for the explanation. Closes #25651 See merge request !8083
| * | Fix wrong error message expectation in API::Commits spec25651-fix-masterRémy Coutable2016-12-141-1/+1
| | | | | | | | | | | | | | | | | | | | | This was because the MR was a bit behind master and the error message got updated in the meantime by 79aad815. Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Merge branch 'process-commit-worker-migration-encoding' into 'master' Rémy Coutable2016-12-143-7/+44
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Encode when migrating ProcessCommitWorker jobs ## What does this MR do? This adds encoding logic to the migration for ProcessCommitWorker, ensuring it doesn't throw errors when the input can not be converted to UTF-8 without extra help. ## What are the relevant issue numbers? https://gitlab.com/gitlab-org/gitlab-ce/issues/25489 See merge request !8064
| * | | Encode when migrating ProcessCommitWorker jobsprocess-commit-worker-migration-encodingYorick Peterse2016-12-133-7/+44
| | |/ | |/| | | | | | | | | | | | | | | | | | | If the source encoding is not UTF-8 we need to encode the data as `JSON.dump` may throw an error if the input can not be converted to UTF-8. We only encode when necessary to reduce the overhead. Fixes gitlab-org/gitlab-ce#25489
* | | Merge branch '25136-last-deployment-link' into 'master' Grzegorz Bizon2016-12-144-4/+46
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix broken link for latest deployment ## What does this MR do? Creates a new helper for the latest deployment link. ## Why was this MR needed? A helper that returns a link was being called inside another helper that creates a link, resulting in `builds/<a href="builds/ID>#id</a>` ## What are the relevant issue numbers? Closes #25136 See merge request !7839
| * | Changes after review25136-last-deployment-linkFilipa Lacerda2016-12-073-24/+19
| | |
| * | Changes after reviewFilipa Lacerda2016-12-073-13/+11
| | | | | | | | | | | | Fix error
| * | Updates testsFilipa Lacerda2016-12-071-12/+30
| | | | | | | | | | | | | | | | | | | | | | | | Fix tests Fix rubocop error Fix broken test
| * | Fix broken link for latest deploymentFilipa Lacerda2016-12-074-2/+33
| | | | | | | | | | | | Updates changelog with MR ID
* | | Merge branch '20052-actions-table-vscroll' into 'master' Annabel Dunstone Gray2016-12-133-13/+16
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prevent overflow with vertical scroll when we have space to show content ## What does this MR do? Removes vertical and horizontal scroll from builds, environments and pipelines tables by adding a media query since this is only necessary in smaller screens. ## Screenshots (if relevant) ![scroll](/uploads/6b9e3584df6e4ca29553d2ff576b794e/scroll.jpg) ## Does this MR meet the acceptance criteria? - [ ] [Changelog entry](https://docs.gitlab.com/ce/development/changelog.html) added - [ ] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [ ] API support added - Tests - [ ] Added for this feature/bug - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Closes #20052 See merge request !8061