summaryrefslogtreecommitdiff
path: root/lib/gitlab/github_import
Commit message (Collapse)AuthorAgeFilesLines
* Whitelist or fix additional `Gitlab/PublicSend` cop violationsrs-more-public-send-whitelistsRobert Speicher2017-08-143-3/+5
| | | | | An upcoming update to rubocop-gitlab-security added additional violations.
* Rename more path_with_namespace -> full_path or disk_pathGabriel Mazetto2017-08-012-3/+3
|
* Rename many path_with_namespace -> full_pathGabriel Mazetto2017-08-011-1/+1
|
* Backport of multiple_assignees_feature [ci skip]Valery Sizov2017-05-041-1/+1
|
* Rename skip_metrics to imported on the importable concernDouglas Barbosa Alexandre2017-04-031-1/+1
|
* Skip MR metrics when importing projects from GitHubDouglas Barbosa Alexandre2017-04-031-1/+2
|
* Fetch GitHub project as a mirror to get all refs at onceDouglas Barbosa Alexandre2017-04-031-1/+1
|
* Minor refactorGabriel Mazetto2017-04-032-12/+7
|
* Fix BrachFormatter for removed usersDouglas Barbosa Alexandre2017-04-031-1/+1
|
* One more change to the branch names to preserve metadataGabriel Mazetto2017-04-031-2/+2
|
* Prefixes source branch name with short SHA to avoid collisionDouglas Barbosa Alexandre2017-04-032-9/+17
|
* Fix GitHub importer for PRs of deleted forked repositoriesGabriel Mazetto2017-04-032-4/+20
|
* Using guard clause and added more specs29046-fix-github-importer-open-prsGabriel Mazetto2017-03-102-9/+3
|
* Fix GitHub Import for open PRs from a forkGabriel Mazetto2017-03-102-1/+13
|
* Refactor some code29034-fix-github-importerGabriel Mazetto2017-03-081-2/+6
|
* Fix name colision when importing GitHub pull requests from forked repositoriesGabriel Mazetto2017-03-081-1/+5
|
* Fix 'Object not found - no match for id (sha)' when importing GitHub PRsDouglas Barbosa Alexandre2017-03-071-1/+1
|
* Revert "Enable Style/DotPosition"Douwe Maan2017-02-232-10/+10
| | | | | | | | | | | | | | | This reverts commit e00fb2bdc2090e9cabeb1eb35a2672a882cc96e9. # Conflicts: # .rubocop.yml # .rubocop_todo.yml # lib/gitlab/ci/config/entry/global.rb # lib/gitlab/ci/config/entry/jobs.rb # spec/lib/gitlab/ci/config/entry/factory_spec.rb # spec/lib/gitlab/ci/config/entry/global_spec.rb # spec/lib/gitlab/ci/config/entry/job_spec.rb # spec/lib/gitlab/ci/status/build/factory_spec.rb # spec/lib/gitlab/incoming_email_spec.rb
* Revert "Prefer leading style for Style/DotPosition"Douwe Maan2017-02-232-10/+10
| | | | This reverts commit cb10b725c8929b8b4460f89c9d96c773af39ba6b.
* Enable Rails/DelegateDouwe Maan2017-02-231-3/+1
|
* Prefer leading style for Style/DotPositionDouwe Maan2017-02-232-10/+10
|
* Enable Performance/RedundantMergeDouwe Maan2017-02-231-1/+1
|
* Enable Style/DotPositionDouwe Maan2017-02-232-10/+10
|
* Reuse User#find_by_any_email on UserFormatter#find_by_emailDouglas Barbosa Alexandre2017-02-201-10/+2
|
* Use leading periods on UserFormatter#find_by_external_uidDouglas Barbosa Alexandre2017-02-201-2/+3
|
* Add a simple cache for Gitlab::GithubImport::Client#userDouglas Barbosa Alexandre2017-02-201-0/+8
|
* GitHub Importer - Find users based on their email addressDouglas Barbosa Alexandre2017-02-205-36/+89
| | | | | | The returned email by the GitHub API is the user's publicly visible email address (or null if the user has not specified a public email address in their profile)
* Merge branch 'fix-github-import-MR-wrong-project' into 'security' Robert Speicher2017-02-151-3/+7
| | | | | Fix labels being applied to wrong merge requests on GitHub import See https://dev.gitlab.org/gitlab/gitlabhq/merge_requests/2064
* Revert "Merge branch 'revert-9cac0317' into 'master' "Grzegorz Bizon2017-01-231-1/+3
| | | | | This reverts commit c20934869f7dc8cfbdbafb6ecb7b1305452c9e8a, reversing changes made to 4b7ec44b91e0571d209c790d54947ba1756dac0e.
* Revert "Merge branch 'dont-persist-application-settings-in-test-env' into ↵Grzegorz Bizon2017-01-211-3/+1
| | | | | 'master'" This reverts merge request !8573
* Don't persist ApplicationSetting in test envdont-persist-application-settings-in-test-envRémy Coutable2017-01-151-1/+3
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Add Project#gitea_import?22348-gitea-importerRémy Coutable2016-12-192-4/+4
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Modify GithubImport to support GiteaRémy Coutable2016-12-197-141/+143
| | | | | | | | The reason is that Gitea plan to be GitHub-compatible so it makes sense to just modify GitHubImport a bit for now, and hopefully we can change it to GitHubishImport once Gitea is 100%-compatible. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Gogs ImporterKim "BKC" Carlbäcker2016-12-196-20/+31
|
* Fix branch validation for GitHub PR where repo/fork was renamed/deletedDouglas Barbosa Alexandre2016-12-011-1/+1
|
* Refactor github import to reduce number of API callsAndrew Smith2016-11-182-25/+29
| | | | Ref #24073
* Cache imported GH labels no matter whatfix/gh-imported-labels-are-not-applied-correctlyAhmad Sherif2016-11-041-6/+3
|
* Fix applying GitHub-imported labels when importing job is interruptedAhmad Sherif2016-11-041-0/+10
| | | | Closes #24075
* Merge branch 'adam-fix-labels-find-or-create' into 'master' Douwe Maan2016-10-281-2/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pass user instance to Labels::FindOrCreateService or skip_authorization: true ## What does this MR do? It fixes a bug described in #23694 when `project.owner` was passed to `Labels::FindOrCreateService`. `Labels::FindOrCreateService` expected a user instance and `project.owner` may return a group as well. This MR makes sure that we either pass a user instance or `skip_authorization: true`. ## Are there points in the code the reviewer needs to double check? - places where we pass `skip_authorization: true` ## Does this MR meet the acceptance criteria? - Tests - [x] Added for this feature/bug - [ ] All builds are passing - [ ] 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? Fixes #23694 See merge request !7093
| * Pass user instance to Labels::FindOrCreateService or skip_authorization: trueadam-fix-labels-find-or-createAdam Niedzielski2016-10-281-2/+2
| | | | | | | | | | | | Do not pass project.owner because it may return a group and Labels::FindOrCreateService throws an error in this case. Fixes #23694.
* | Fix typosfix/make-github-import-retryableAhmad Sherif2016-10-282-2/+2
| |
* | Abstract the use of imported[!?] and {current,increment}_page in GitHub importerAhmad Sherif2016-10-281-50/+34
| |
* | Use public_send instead of sendAhmad Sherif2016-10-281-1/+1
| |
* | Modify GitHub importer to be retryableAhmad Sherif2016-10-287-25/+130
| |
* | Fix importing MR comments from GitHubfix/gh-import-bugsAhmad Sherif2016-10-271-6/+7
| |
* | Fix applying labels for GitHub-imported MRsAhmad Sherif2016-10-272-7/+16
|/
* Rename Labels::CreateService to Labels::FindOrCreateServiceDouglas Barbosa Alexandre2016-10-191-1/+1
|
* Fix GitHub importer specDouglas Barbosa Alexandre2016-10-191-1/+4
|
* User Labes::CreateService to create labelsDouglas Barbosa Alexandre2016-10-191-3/+4
|
* Fix broken handling of certain calls in GitHub importer clientfix/github-importer-clientAhmad Sherif2016-10-061-2/+12
| | | | Closes #22998