summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* refactor emails servicerefactor-servicesJames Lopez2017-09-281-4/+4
|
* refactor users update serviceJames Lopez2017-09-285-5/+5
|
* refactor services to match EE signatureJames Lopez2017-09-285-9/+9
|
* Merge branch 'backstage/gb/refactor-pipeline-create-service' into 'master'Kamil Trzciński2017-09-2710-141/+412
|\ | | | | | | | | | | | | Refactor a service responsible for creating a pipeline Closes #37563 and #34415 See merge request gitlab-org/gitlab-ce!14482
| * Add some minor improvements to pipeline creation chainbackstage/gb/refactor-pipeline-create-serviceGrzegorz Bizon2017-09-272-5/+2
| |
| * Add specs for builder chain step that skipps pipelinesGrzegorz Bizon2017-09-261-1/+1
| |
| * Fix coding style offenses in pipeline chain classesGrzegorz Bizon2017-09-262-4/+2
| |
| * Move pipeline builder validation chain to a moduleGrzegorz Bizon2017-09-266-115/+121
| |
| * Split pipeline chain builder validation classGrzegorz Bizon2017-09-266-102/+143
| |
| * Add class that handles pipeline creation sequenceGrzegorz Bizon2017-09-261-0/+37
| |
| * Extract pipeline persistence to a separate chain classGrzegorz Bizon2017-09-261-0/+29
| |
| * Fix code style offenses in pipeline create servicesGrzegorz Bizon2017-09-261-1/+1
| |
| * Use tag/branch methods to check if pipeline ref existsGrzegorz Bizon2017-09-251-8/+2
| |
| * Extract pipeline chain builder classes from serviceGrzegorz Bizon2017-09-253-0/+167
| |
| * Move related pipeline class to new pipeline moduleGrzegorz Bizon2017-09-252-141/+143
| |
* | Merge branch ↵Rémy Coutable2017-09-271-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | '37467-helper-method-from-users-endpoint-overrides-api-helper-method' into 'master' find_user users helper method no longer overrides find_user API helper method. Closes #37467 See merge request gitlab-org/gitlab-ce!14418
| * | find_user users helper method no longer overrides find_user API helper method.37467-helper-method-from-users-endpoint-overrides-api-helper-methodTiago Botelho2017-09-261-2/+2
| | |
* | | Add a spec for Github::Client and revert an `if !` to `unless`38198-fetch-github-api-per-100Rémy Coutable2017-09-261-1/+1
| | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Retrieve PR comments only when we know there are anyRémy Coutable2017-09-262-20/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Pull Request resource doesn't include the `comments` field, while the Issue resource does. And since we're looping through all issues anyway, we can freely check if the issue is a PR and has comments and in this case only fetch comments for it. That means if you have 1000 PRs but only 200 with comments, you will do 200 API requests instead of 1000. :notbad: Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Fetch 100 results when calling the GitHub API in Github::ImportRémy Coutable2017-09-261-6/+6
| | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Merge branch 'dm-simple-project-avatar-url' into 'master'Rémy Coutable2017-09-261-5/+6
|\ \ \ | | | | | | | | | | | | | | | | Expose avatar_url when requesting list of projects from API with simple=true See merge request gitlab-org/gitlab-ce!14498
| * | | Expose avatar_url when requesting list of projects from API with simple=truedm-simple-project-avatar-urlDouwe Maan2017-09-261-5/+6
| | | |
* | | | Merge branch 'dm-bitbucket-import-truncated-shas' into 'master'Robert Speicher2017-09-263-10/+32
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug that caused merge requests with diff notes imported from Bitbucket to raise errors Closes #38100 See merge request gitlab-org/gitlab-ce!14438
| * | | | Remove unnecessary commentsDouwe Maan2017-09-251-2/+2
| | | | |
| * | | | Properly compare diff refs and diff positions when shas are truncatedDouwe Maan2017-09-252-8/+25
| | | | |
| * | | | Expand truncated Bitbucket pull request source/target SHA if possibleDouwe Maan2017-09-251-2/+7
| | |/ / | |/| |
* | | | Expose last pipeline details in API response when getting a single commitMehdi Lahmam2017-09-261-0/+1
| |/ / |/| | | | | | | | Closes #35692.
* | | Remove dead `Gitlab::Markdown::Pipeline` classrs-remove-dead-markdown-pipelineRobert Speicher2017-09-251-32/+0
| |/ |/|
* | Migrate Gitlab::Git::Repository#log to GitalyKim "BKC" Carlbäcker2017-09-253-1/+35
|/
* Merge branch '38280-undefined-run_command-when-running-rake-gitlab-check' ↵Douwe Maan2017-09-252-2/+2
|\ | | | | | | | | | | | | | | | | into 'master' Resolve "Undefined run_command when running rake gitlab:check" Closes #38280 See merge request gitlab-org/gitlab-ce!14469
| * Fixed few gitlab:check tasks that were failing with exception38280-undefined-run_command-when-running-rake-gitlab-checkGabriel Mazetto2017-09-252-2/+2
| |
* | Merge branch 'rs-allow-name-on-anchors' into 'master'Douwe Maan2017-09-251-1/+2
|\ \ | |/ |/| | | | | | | | | Re-allow `name` attribute on user-provided anchor HTML Closes #38196 See merge request gitlab-org/gitlab-ce!14452
| * Re-allow `name` attribute on user-provided anchor HTMLRobert Speicher2017-09-221-1/+2
| |
* | Merge branch '36549-circuit-breaker-handles-missing-storages' into 'master'Douwe Maan2017-09-225-10/+72
|\ \ | | | | | | | | | | | | | | | | | | Allow the git circuit breaker to correctly handle missing repository storages Closes #36549 See merge request gitlab-org/gitlab-ce!14417
| * | Allow the git circuit breaker to correctly handle missing repository storagesNick Thomas2017-09-225-10/+72
| | |
* | | Workaround for #38259Andrew Newdigate2017-09-221-2/+5
| |/ |/|
* | Merge branch 'feature/migrate-repository-diff' into 'master'Douwe Maan2017-09-224-15/+60
|\ \ | | | | | | | | | | | | | | | | | | Migrate Gitlab::Git::Repository#diff to Gitaly Closes gitaly#524 See merge request gitlab-org/gitlab-ce!14251
| * | Migrate Gitlab::Git::Repository#diff to Gitalyfeature/migrate-repository-diffAhmad Sherif2017-09-214-15/+60
| | | | | | | | | | | | Closes gitaly#524
* | | Merge branch '38197-fix-ImapAuthenticationCheck' into 'master'Douwe Maan2017-09-221-17/+28
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix `rake gitlab:incoming_email:check` Closes #38197 See merge request gitlab-org/gitlab-ce!14423
| * | | Fix rake gitlab:incoming_email:check and make it38197-fix-ImapAuthenticationCheckLin Jen-Shin2017-09-221-17/+28
| | | | | | | | | | | | | | | | report error properly, so that we know what's really wrong.
* | | | Merge branch '33328-usage-ping-for-gitlab-features-and-components' into 'master'Rémy Coutable2017-09-221-10/+36
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds gitlab features and components to usage ping data. Closes #33328 See merge request gitlab-org/gitlab-ce!14305
| * | | | Adds gitlab features and components to usage ping data.33328-usage-ping-for-gitlab-features-and-componentsTiago Botelho2017-09-211-10/+36
| | | | |
* | | | | Merge branch 'sh-blob-raw-check' into 'master'Douwe Maan2017-09-221-0/+2
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | Ensure that Blob.raw returns always returns a valid blob object See merge request gitlab-org/gitlab-ce!14412
| * | | | Ensure that Blob.raw returns always returns a valid blob objectsh-blob-raw-checkStan Hu2017-09-211-0/+2
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | In gitlab-org/gitlab-ee!2976, we saw that a given OID could point to a commit, which would cause the delta size check to fail. Gitaly already returns nil if the OID isn't a blob, so this change makes the Rugged implementation consistent.
* | | | Resolve "Better SVG Usage in the Frontend"Tim Zallmann2017-09-221-0/+1
| | | |
* | | | Merge branch 'fix-multi-line-hook-output' into 'master'Rémy Coutable2017-09-221-3/+4
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | Display full pre-receive and post-receive hook output in GitLab UI Closes #25214 See merge request gitlab-org/gitlab-ce!14222
| * | | Display full pre-receive and post-receive hook output in GitLab UIRobin Bobbitt2017-09-211-3/+4
| |/ /
* | | Merge branch 'bvl-transient-failure-health-controller-spec' into 'master'Douwe Maan2017-09-211-2/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Don't memoize storage configuration on `FsShardsCheck` Closes #38140 See merge request gitlab-org/gitlab-ce!14419
| * | | Don't memoize storage configuration on `FsShardsCheck`bvl-transient-failure-health-controller-specBob Van Landuyt2017-09-211-2/+2
| |/ / | | | | | | | | | | | | | | | | | | | | | These values are already coming from memory, so we don't need to memoize. If we do memoize these, than the would not be stubbed when calling `stub_storage_settings` in specs
* | | Merge branch 'dm-closing-issue-urls' into 'master'Robert Speicher2017-09-211-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Correctly detect multiple issue URLs after 'Closes...' in MR descriptions Closes #33745 See merge request gitlab-org/gitlab-ce!14411