summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | Add spec for more examples of trailing punctuation that should be ignoreddm-label-reference-periodDouwe Maan2018-06-201-3/+5
| | | | | | | | |
| * | | | | | | | Properly detect label reference if followed by period or question markDouwe Maan2018-06-183-7/+17
| | | | | | | | |
* | | | | | | | | Merge branch 'blackst0ne-rails5-fix-action-cable-mount-path' into 'master'Rémy Coutable2018-06-211-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Rails5] Fix ActionCable's mount_path configuration See merge request gitlab-org/gitlab-ce!20051
| * | | | | | | | | [Rails5] Fix ActionCable's mount_path configurationblackst0ne-rails5-fix-action-cable-mount-pathblackst0ne2018-06-211-1/+1
| | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original MR [1] fixed red specs for Rails 5. But while the failed specs were fixed, that changes brought new failed specs which weren't caught up in [1]. This commit just fixes the fix. :) New errors are like these: ``` Failed to upgrade to WebSocket (REQUEST_METHOD: GET, HTTP_CONNECTION: keep-alive, HTTP_UPGRADE: ) Finished "/-/boards/1/lists"[non-WebSocket] for 127.0.0.1 at 2018-06-20 18:09:26 +0200 ``` [1]: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/20015
* | | | | | | | | Merge branch 'rails5-fix-48140' into 'master'Rémy Coutable2018-06-213-0/+10
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rails 5 fix Capybara::ElementNotFound: Unable to find visible css #modal-revert-commit Closes #48140 et #48139 See merge request gitlab-org/gitlab-ce!20044
| * | | | | | | | | Rails 5 fix Capybara::ElementNotFound: Unable to find visible css ↵Jasper Maes2018-06-203-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | #modal-revert-commit and expected: "/bar" got: "/foo"
* | | | | | | | | | Merge branch 'zj-raw-changes-mandatory' into 'master'Douwe Maan2018-06-212-54/+29
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Raw changes calculation is Gitaly only now Closes gitaly#1138 See merge request gitlab-org/gitlab-ce!20057
| * | | | | | | | | | Raw changes calculation is Gitaly only nowZeger-Jan van de Weg2018-06-212-54/+29
| | |_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes https://gitlab.com/gitlab-org/gitaly/issues/1138
* | | | | | | | | | Merge branch 'bvl-finder-methods-cop' into 'master'Douwe Maan2018-06-215-4/+110
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a cop for `FinderMethods` Closes #42705 See merge request gitlab-org/gitlab-ce!19951
| * | | | | | | | | Add a cop for `FinderMethods`bvl-finder-methods-copBob Van Landuyt2018-06-205-4/+110
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This notifies developers when calling `find(_by!)` chained on `execute`. And suggests using the methods from `FinderMethods`. These will perform the correct authorization checks on the resource when it is found.
* | | | | | | | | | Update VERSION to 11.1.0-prev11.1.0.preFilipa Lacerda2018-06-211-1/+1
| | | | | | | | | |
* | | | | | | | | | Update CHANGELOG.md for 11.0.0Filipa Lacerda2018-06-21214-1074/+240
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | | | | Merge branch 'MadLittleMods-master-patch-55656' into 'master'Filipa Lacerda2018-06-211-2/+2
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix a couple Vuex typos See merge request gitlab-org/gitlab-ce!20046
| * | | | | | | | | | Fix a couple Vuex typosMadLittleMods-master-patch-55656Eric Eastwood2018-06-201-2/+2
| | | | | | | | | | |
* | | | | | | | | | | Merge branch 'docs-styleguidelines-nav' into 'master'Achilleas Pipinellis2018-06-211-2/+16
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Docs: update style guidelines with rules for navigation items See merge request gitlab-org/gitlab-ce!20024
| * | | | | | | | | | | clarify what "methods" aredocs-styleguidelines-navMarcia Ramos2018-06-201-3/+4
| | | | | | | | | | | |
| * | | | | | | | | | | Update styleguide.md with rules for navigation itemsMarcia Ramos2018-06-201-2/+15
| | | | | | | | | | | |
* | | | | | | | | | | | Merge branch 'mk/fix-n-plus-1-queries-in-uploads-check-rake-task' into 'master'Stan Hu2018-06-203-9/+39
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix N+1 queries in uploads check rake task Closes #47924 See merge request gitlab-org/gitlab-ce!20012
| * | | | | | | | | | | Fix documentation code blocksmk/fix-n-plus-1-queries-in-uploads-check-rake-taskMichael Kozono2018-06-201-0/+1
| | | | | | | | | | | |
| * | | | | | | | | | | Avoid N+1 queries while processing uploadsMichael Kozono2018-06-202-9/+38
| | | | | | | | | | | |
* | | | | | | | | | | | Merge branch 'sh-fallback-to-first-tab-signin' into 'master'Clement Ho2018-06-202-0/+20
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fall back to the first sign-in tab if the local storage value is bad See merge request gitlab-org/gitlab-ce!20050
| * | | | | | | | | | | | Fall back to the first sign-in tab if the local storage value is badStan Hu2018-06-202-0/+20
| | |_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If `current_signin_tab` is `#crowd`, then no tab will be active. Fix this by using the first available tab. Relates to https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/20049#note_82996324
* | | | | | | | | | | | Merge branch 'bvl-dont-generate-mo' into 'master'Rubén Dávila2018-06-202-1/+5
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't generate `.mo` files Closes #47979 See merge request gitlab-org/gitlab-ce!20048
| * | | | | | | | | | | | Don't generate `.mo` filesBob Van Landuyt2018-06-202-1/+5
| | |/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we are only using `.po` files for translation we can shave off some setup time. Generating the `mo` files also regenerates the PO-files, including fuzzy translations. Those cause a lot of bugs, and we don't need them anymore since the `po` files are generated by an external translation service.
* | | | | | | | | | | | Merge branch 'jr-48133-web-ide-commit-ellipsis' into 'master'Tim Zallmann2018-06-202-1/+6
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolve "Web IDE "Commit" button should indicate when there's a subsequent step" Closes #48133 See merge request gitlab-org/gitlab-ce!20030
| * | | | | | | | | | | Add ellipsis to Web IDE commit buttonjr-48133-web-ide-commit-ellipsisJames Ramsay2018-06-202-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The blue shortcut commit button opens the commit interface to actually perform the commit. The ellipsis indicates the button doesn't immediately perform the commit.
* | | | | | | | | | | | Merge branch 'improve-scripts-trigger-build' into 'master'Robert Speicher2018-06-201-0/+4
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ignore GitLab API hiccups in scripts/trigger-build See merge request gitlab-org/gitlab-ce!20038
| * | | | | | | | | | | | Ignore GitLab API hiccups in scripts/trigger-buildimprove-scripts-trigger-buildRémy Coutable2018-06-201-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | | | | | | | | | Merge branch ↵Robert Speicher2018-06-202-2/+10
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '48159-qa-specs-features-login-ldap_spec-rb-is-failing-after-https-gitlab-com-gitlab-org-gitlab-ce-merge_requests-19964' into 'master' Resolve "qa/specs/features/login/ldap_spec.rb is failing after https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/19964" Closes #48159 See merge request gitlab-org/gitlab-ce!20027
| * | | | | | | | | | | | | [QA] Fix qa/qa/specs/features/login/ldap_spec.rb48159-qa-specs-features-login-ldap_spec-rb-is-failing-after-https-gitlab-com-gitlab-org-gitlab-ce-merge_requests-19964Rémy Coutable2018-06-202-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | | | | | | | | | | Merge branch 'unify-lib-tasks' into 'master'Robert Speicher2018-06-202-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unify flay.rake and gettext.rake from EE See merge request gitlab-org/gitlab-ce!20041
| * | | | | | | | | | | | | | Unify flay.rake and gettext.rake from EEunify-lib-tasksLin Jen-Shin2018-06-212-2/+2
| | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | Merge branch '47488-remove-unmergeable-notification-for-no-commits' into ↵Robert Speicher2018-06-209-70/+33
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Notify only merge request unmergeable due to conflict See merge request gitlab-org/gitlab-ce!19548
| * | | | | | | | | | | | | | Fix possiblilty of branch not found47488-remove-unmergeable-notification-for-no-commitsMark Chao2018-06-201-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This now can happen because can_be_merged? is called during MR merge_status transition to cannot_be_merged. It is possible branch_name is invalid.
| * | | | | | | | | | | | | | Notify only when unmergeable due to conflictMark Chao2018-06-208-68/+28
| | |_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is still the edge case when 'no commits' changes to 'conflict' would not trigger notification, which we ignore for now. Calling can_be_merged? can cause exception (e.g. non-UTF8) Ignore those by rescueing. Remove unmergeable_reason as now only conflict is notified Update spec
* | | | | | | | | | | | | | Merge branch 'project_wiki_scenario_automation' into 'master'Rémy Coutable2018-06-2022-70/+314
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added Wiki Scenario Automation gitlab-qa#263 Closes gitlab-qa#263 See merge request gitlab-org/gitlab-ce!19568
| * | | | | | | | | | | | | Added Wiki Scenario Automation gitlab-qa#263Ramya A2018-06-2022-70/+314
|/ / / / / / / / / / / / /
* | | | | | | | | | | | | Merge branch '47964-custom-logo-rendering-improperly-on-sign-in-page-2' into ↵Mike Greiling2018-06-201-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Resolve "Custom logo rendering improperly on sign-in page" Closes #47964 See merge request gitlab-org/gitlab-ce!20037
| * | | | | | | | | | | | | Fix navigation bar center logo for custom logosClement Ho2018-06-201-1/+1
| |/ / / / / / / / / / / /
* | | | | | | | | | | | | Merge branch 'rs-platform-feature-spec-parity' into 'master'Rémy Coutable2018-06-201-2/+7
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the stubbed methods for SAML authorize paths more specific See merge request gitlab-org/gitlab-ce!20009
| * | | | | | | | | | | | | Make the stubbed methods for SAML authorize paths more specificrs-platform-feature-spec-parityRobert Speicher2018-06-191-2/+7
| | |_|_|_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Globally stubbing any instance of `Object` seems like a very bad idea, and it led to test failures here: - https://gitlab.com/gitlab-org/gitlab-ee/-/jobs/76025159 - https://gitlab.com/gitlab-org/gitlab-ee/-/jobs/76024972
* | | | | | | | | | | | | Merge branch '6165-migrate-ee-specific-code' into 'master'Rémy Coutable2018-06-202-10/+55
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Backport code from gitlab-ee!6182 See merge request gitlab-org/gitlab-ce!20001
| * | | | | | | | | | | | | Backport code from gitlab-ee!6182Lukas Eipert2018-06-202-10/+55
|/ / / / / / / / / / / / /
* | | | | | | | | | | | | Merge branch 'limit-metrics-content-type' into 'master'Stan Hu2018-06-203-3/+22
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Limit the action suffixes in transaction metrics See merge request gitlab-org/gitlab-ce!20032
| * | | | | | | | | | | | | Limit the action suffixes in transaction metricslimit-metrics-content-typeYorick Peterse2018-06-203-3/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There seem to be a lot of cases where the suffix of an action (e.g. ".html") is set to bogus data, such as "*/*" or entire URLs. This can increase cardinality of our metrics, and isn't very useful for monitoring and filtering. To work around this, we enforce a whitelist containing a few suffixes we actually care about. Suffixes not supported will be grouped under the action without a suffix. This means that a request to "FooController#bar.jpeg" will be assigned to "FooController#bar".
* | | | | | | | | | | | | | Merge branch 'remove-allocations-gem' into 'master'Stan Hu2018-06-208-29/+5
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove remaining traces of the Allocations Gem See merge request gitlab-org/gitlab-ce!20029
| * | | | | | | | | | | | | Remove remaining traces of the Allocations Gemremove-allocations-gemYorick Peterse2018-06-208-29/+5
| |/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In MR https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15834 we removed use of the data produced by the Allocations Gem. However, we never removed the code that just enables tracking of allocations. In this commit we remove all remaining traces of this Gem.
* | | | | | | | | | | | | Merge branch 'highlight-cluster-settings-message' into 'master'Rémy Coutable2018-06-202-2/+8
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Highlight cluster settings message Closes #41274 See merge request gitlab-org/gitlab-ce!19996
| * | | | | | | | | | | | | Highlight cluster settings messageGeorge Tsiolis2018-06-192-2/+8
| | | | | | | | | | | | | |
* | | | | | | | | | | | | | Merge branch ↵Rémy Coutable2018-06-202-2/+7
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '48153-date-selection-dialog-broken-when-creating-a-new-milestone' into 'master' Resolve "Date selection dialog broken when creating a new milestone" Closes #48153 See merge request gitlab-org/gitlab-ce!20033