summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Simplify entities for branches and tags APIapi-cleanup-entitiesRobert Schilling2016-07-192-42/+21
|
* Merge branch 'new-issue-bg-color' into 'master' Dmitriy Zaporozhets2016-07-191-1/+1
|\ | | | | | | | | | | | | Change color of today issue highlight See merge request !5329
| * Change color of today issue highlightnew-issue-bg-colortauriedavis2016-07-181-1/+1
| |
* | Merge branch 'api-dev-can-push' into 'master' Rémy Coutable2016-07-197-10/+161
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | API: Expose 'developers_can_push' for branches ## What does this MR do? Adds support for the `developers_can_push` flag for the branches API. It also supports creating protected branches with that flag. ## Are there points in the code the reviewer needs to double check? The API call requires an optional boolean parameter `developers_can_push`. If it is not either `true` or `false`, it should not update the value. I'm not sure if this is the right way to do it. Right now it always returns `200`. Maybe the `PUT` method should return different status codes depending if it really created or updated a branch. ## What are the relevant issue numbers? Closes #12735 ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [x] API support added - Tests - [x] Added for this feature/bug - [x] All builds are passing - [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 you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5208
| * | Only update onceRobert Schilling2016-07-191-2/+3
| | |
| * | API: Expose 'developers_can_merge' for branchesRobert Schilling2016-07-195-11/+57
| | |
| * | API: Expose 'developers_can_push' for branchesRobert Schilling2016-07-197-10/+114
| | |
* | | Merge branch 'fix/import-export-encoding' into 'master' James Lopez2016-07-198-13/+44
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Fix MR diff encoding issues exporting projects Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/19338 See merge request !5039
| * | Squashed - fix encoding issuefix/import-export-encodingJames Lopez2016-07-198-13/+44
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | WIP - trying to replicate UTF-8 error fix spec fixing encoding issue and another spec, to do with MR diffs fix issue and spec failure Add changelog and bumped up I/E version fix spec based on feedback - omitted target project
* | Merge branch 'dz-small-ui-improvements' into 'master' Dmitriy Zaporozhets2016-07-192-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | Small ui improvements * make file tree row 2px smaller for more compact view * make sub navigation background a bit darker for better contrast with main navigation cc @hazelyang @alfredo1 See merge request !5315
| * | Make repository files list UI a bit more compactdz-small-ui-improvementsDmitriy Zaporozhets2016-07-181-1/+1
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Make sub navigation background color darker for better visual separationDmitriy Zaporozhets2016-07-181-1/+1
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | Merge branch '19892-documentation' into 'master' Achilleas Pipinellis2016-07-191-1/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update documentation for build artifact dependencies ## What does this MR do? Update documentation for build artifact dependencies ## Are there points in the code the reviewer needs to double check? ## Why was this MR needed? #19892 ## What are the relevant issue numbers? #19892 ## Screenshots (if relevant) See merge request !5332
| * | | Update documentation for build artifact dependencies19892-documentationMark Pundsack2016-07-181-1/+2
| | |/ | |/|
* | | Merge branch 'update-maintenance-md' into 'master' Achilleas Pipinellis2016-07-191-8/+28
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Update MAINTENANCE.md with our latest policy Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/19958 See merge request !5339
| * | | Update MAINTENANCE.md with our latest policyupdate-maintenance-mdAchilleas Pipinellis2016-07-191-8/+28
| | | | | | | | | | | | | | | | [ci skip]
* | | | Merge branch 'backport_from_ee_easy_commit_creation' into 'master' Valery Sizov2016-07-193-59/+43
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix of 'Commits being passed to custom hooks are already reachable when using the UI' This is back-port of https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/563 ## What does this MR do? Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/19771 This MR changes the way how we create commit, we create it directly in the branch and we don't use temporary ref anymore. This also works similar to how `git-receive-pack` works so it's a natural way to create commit. ## What are the relevant issue numbers? https://gitlab.com/gitlab-org/gitlab-ce/issues/19771 See merge request !5337
| * | | | Fix of 'Commits being passed to custom hooks are already reachable when ↵Valery Sizov2016-07-193-59/+43
|/ / / / | | | | | | | | | | | | using the UI'
* | | | Merge branch 'cs-gemojione-3' into 'master' Robert Speicher2016-07-1814-7047/+11353
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Upgrade gemojione to 3.0.0 Upgrades gemojione to 3.0.0, see the Changelog: https://github.com/jonathanwiesel/gemojione/blob/b98aa8b07eef815d4d3f52ff3c8714b28932b0de/CHANGELOG.md#v300-2016-07-12 [Here are all the new 2016 emoji](http://emojione.com/releases/2.2.4/) :tada: This update has new emoji and more sensical categories. See merge request !5237
| * | | | Upgrade Gemojione from 2.6.1 to 3.0.1.Connor Shea2016-07-1814-7047/+11353
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the 2016 emoji as well as support for using SVG images instead of PNGs. It also fixes a number of incorrectly categorized emoji and other minor issues. Upgrade Rake task for Gemojione 3.0.0 and generate sprites. Upgrade aliases.json by pulling down index.json from the gemojione repository and running the generate_aliases.rb file. Changelog: https://github.com/jonathanwiesel/gemojione/blob/master/CHANGELOG.md#v301-2016-07-16 For the specific emoji added to the Unicode standard, see: http://emojione.com/releases/2.2.4/ Huge kudos to Jonathan Wiesel (@jonathanwiesel) for his work on the gemojione gem!
* | | | | Merge branch 'mr-widget-alignment' into 'master' Fatih Acet2016-07-181-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed the alignment in the MR widget ## What does this MR do? Fixes the alignment of the browse button in the MR widget ## Screenshots (if relevant) ![Screen_Shot_2016-07-17_at_17.47.54](/uploads/41dcb0fad38fdd4dbccfda5ffaeaec6e/Screen_Shot_2016-07-17_at_17.47.54.png) See merge request !5307
| * | | | | Fixed the alignment in the MR widgetmr-widget-alignmentPhil Hughes2016-07-171-1/+1
| | | | | |
* | | | | | Merge branch 'sidebar-dropdowns-blur-link' into 'master' Fatih Acet2016-07-181-0/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Blurs the clicked link in dropdown menus ## What does this MR do? This fixes an issue where the dropdown menu would stay visible even when clicked out of it See merge request !5322
| * | | | | | Blurs the clicked link in dropdown menussidebar-dropdowns-blur-linkPhil Hughes2016-07-181-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes an issue where the dropdown menu would stay visible even when clicked out of it
* | | | | | | Merge branch 'bug/sidekiq-cron-better-error' into 'master' Gabriel Mazetto2016-07-182-1/+9
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve cron_jobs loading error messages ## What does this MR do? Improves the error message from "Sidekiq Cron Jobs" plugin, when trying to load a incorrect definition from gitlab config files. This is the output before the fix: ``` 2016-06-08T16:43:19.277Z 2890 TID-gso49ojro INFO: Booting Sidekiq 4.1.2 with redis options {:url=>"unix:/var/run/redis/redis.sock", :namespace=>"resque:gitlab"} undefined method `delete' for nil:NilClass /home/git/gitlab/config/initializers/sidekiq.rb:16:in `block (2 levels) in <top (required)>' /home/git/gitlab/config/initializers/sidekiq.rb:16:in `each' /home/git/gitlab/config/initializers/sidekiq.rb:16:in `block in <top (required)>' /home/git/gitlab/vendor/bundle/ruby/2.1.0/gems/sidekiq-4.1.2/lib/sidekiq.rb:70:in `configure_server' ``` This is the output after the fix: ``` 2016-07-18T10:53:22.414Z 66332 TID-oxal1bmik INFO: Booting Sidekiq 4.1.4 with redis options {:url=>"redis://localhost:6378", :namespace=>"resque:gitlab"} Invalid cron_jobs config key: 'registry'. Check your gitlab config file. /home/git/gitlab/config/initializers/sidekiq.rb:21:in `block (2 levels) in <top (required)>' /home/git/gitlab/config/initializers/sidekiq.rb:17:in `each' /home/git/gitlab/config/initializers/sidekiq.rb:17:in `block in <top (required)>' /home/git/gitlab/vendor/bundle/ruby/2.1.0/gems/ruby-2.1.8/gems/sidekiq-4.1.4/lib/sidekiq.rb:70:in `configure_server' ``` Fixes #18378 ## Are there points in the code the reviewer needs to double check? No ## Why was this MR needed? Original error doesn't give any hint on where the problem can be, so the user can't easily troubleshoot unless he knows implementation details. ## What are the relevant issue numbers? #18378 ## Does this MR meet the acceptance criteria? - [ ] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry 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 - [ ] All builds are passing - [ ] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [ ] Branch has no merge conflicts with `master` (if you do - rebase it please) - [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5318
| * | | | | | | Improve cron_jobs loading error messagesGabriel Mazetto2016-07-182-1/+9
|/ / / / / / /
* | | | | | | Merge branch 'caironoleto/gitlab-ce-cn-issue-19747'Douwe Maan2016-07-188-8/+84
|\ \ \ \ \ \ \
| * | | | | | | Allow bulk (un)subscription from issues in issue indexCairo Noleto2016-07-148-8/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixies #19747
* | | | | | | | Merge branch 'issue_13343' into 'master' Robert Speicher2016-07-187-2/+79
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow build email service to be tested Fixes #13343 See merge request !5223
| * | | | | | | | Allow build email service to be testedissue_13343Felipe Artur2016-07-157-2/+79
| | | | | | | | |
* | | | | | | | | Merge branch 'csp-basics' into 'master' Robert Speicher2016-07-183-0/+116
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Content Security Policy Ref: #18231 This uses twitter's [secure_headers gem](https://github.com/twitter/secureheaders). See merge request !4770
| * | | | | | | | | Minor policy refinements.csp-basicsConnor Shea2016-07-181-8/+12
| | | | | | | | | |
| * | | | | | | | | Document the CSP file.Connor Shea2016-07-181-11/+38
| | | | | | | | | |
| * | | | | | | | | Only enable CSP policies when relevant features are enabled.Connor Shea2016-07-181-1/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Gravatar, Google Analytics, Piwik, Recaptcha, etc.
| * | | | | | | | | Remove background_jobs-specific headers.Connor Shea2016-07-182-7/+0
| | | | | | | | | |
| * | | | | | | | | Only report to Sentry when it's enabled.Connor Shea2016-07-181-2/+6
| | | | | | | | | |
| * | | | | | | | | Add Sidekiq-specific headers.Connor Shea2016-07-182-6/+12
| | | | | | | | | |
| * | | | | | | | | Add the CSP reporting URI of Sentry.Connor Shea2016-07-181-1/+8
| | | | | | | | | |
| * | | | | | | | | Update image policy to allow external images over HTTPS.Connor Shea2016-07-181-1/+1
| | | | | | | | | |
| * | | | | | | | | Remove unsafe eval directive from scripts.Connor Shea2016-07-181-1/+1
| | | | | | | | | |
| * | | | | | | | | Fix that which hath been broken. Except the sidekiq admin iframe.Connor Shea2016-07-182-0/+42
| | | | | | | | | |
| * | | | | | | | | Initial secure_headers config after some testing.Connor Shea2016-07-182-0/+7
| | |_|_|_|_|_|/ / | |/| | | | | | |
* | | | | | | | | Merge branch '19820-safer-diffs' into 'master' Douwe Maan2016-07-1812-23/+96
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Collapsed diffs lines/size don't accumulate to overflow diffs. ## What does this MR do? Reduce the number of lines that we highlight on big diffs to try to reduce the degradation introduced by !4990 as you can see on the issue description #19820 . We collapse any files after the diff is over the number of safe files (100), or over the safe lines (500) or over the safe number of bytes (5KB x 100 files). We still need to bump the gitlab_git and point this branch to the new gitlab_git version. ## What are the relevant issue numbers? Closes #19820 Closes #19885 ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - ~~[ ] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)~~ - ~~[ ] API support added~~ - Tests - [x] Added for this feature/bug - [x] All builds are passing - [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 you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !5306
| * | | | | | | | | Collapsed diffs lines/size don't accumulate to overflow diffs.19820-safer-diffsPaco Guzman2016-07-1812-23/+96
| | | | | | | | | |
* | | | | | | | | | Merge branch '19420-render-only-commit-title' into 'master' 19889-shortcuts-modal-is-brokenDouwe Maan2016-07-184-4/+24
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Render only commit title ## What does this MR do? Render only commit title in builds ## Why was this MR needed? Better readability ## What are the relevant issue numbers? Closes #19420 ## Screenshots (if relevant) ![after](/uploads/fcdd59d74a0f91f6ac847f7f02d5f07f/after.png)![before](/uploads/71ca6dabf31452a1a8dfb15ecb8860c5/before.png) ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - Tests - [x] All builds are passing - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) cc: @kradydal @yorickpeterse @grzesiek @tmaczukin @dzaporozhets [@tomash](https://github.com/tomash) [@chastell](https://github.com/chastell) this is connected to closed MR [!5109](https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5109) See merge request !5131
| * | | | | | | | | add modification to commit title testubudzisz2016-07-181-9/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add modification to commit title test add empty line
| * | | | | | | | | modify view test to commit titleubudzisz2016-07-182-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | modify view test to commit title-repair rubocop modify view test to commit title-repair rubocop modify view test to commit title-repair rubocop remove empty line in CHANGELOG
| * | | | | | | | | Update CHANGELOGula budziszewska2016-07-181-2/+1
| | | | | | | | | |
| * | | | | | | | | add test to viewubudzisz2016-07-182-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | repair rubocop test and update CHANGELOG repair rubocop test repair rubocop repair rubocop repair rubocop
| * | | | | | | | | render only commit titleubudzisz2016-07-181-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | update CHANGELOG