summaryrefslogtreecommitdiff
path: root/lib/github
Commit message (Collapse)AuthorAgeFilesLines
* Replace old GH importer with the parallel importergithub-importer-refactorYorick Peterse2017-11-0723-947/+0
|
* Merge branch 'remove-ensure-ref-fetched-from-controllers' into 'master'Sean McGivern2017-11-061-1/+0
|\ | | | | | | | | | | | | removed the #ensure_ref_fetched from all controllers Closes #36061 See merge request gitlab-org/gitlab-ce!15129
| * removed the #ensure_ref_fetched from all controllersmicael.bergeron2017-11-031-1/+0
| | | | | | | | | | | | | | | | also, I refactored the MergeRequest#fetch_ref method to express the side-effect that this method has. MergeRequest#fetch_ref -> MergeRequest#fetch_ref! Repository#fetch_source_branch -> Repository#fetch_source_branch!
* | Encapsulate git operations for mirroring in Gitlab::GitAlejandro Rodríguez2017-11-031-1/+3
|/
* Use wiki.disk_pathNick Thomas2017-10-181-1/+1
|
* Move line code generation into Gitlab::Gitconflict-resolution-refactorAlejandro Rodríguez2017-10-121-1/+1
| | | | Having a distinct class just for that was a bit overkill
* Create a Gitlab::Git submodule for conlict-related filesAlejandro Rodríguez2017-10-121-1/+1
| | | | Rename classes to (hopefully) clearer names while we're doing that.
* Move Gitlab::Diff::LineCode to module Gitlab::GitAlejandro Rodríguez2017-10-121-1/+1
|
* Minor improvements after latest reviewRémy Coutable2017-10-052-6/+2
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Improve & simplify GitHub's representation classesRémy Coutable2017-10-054-17/+15
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Set `label_ids` and `assignee_ids` after the initial `save!`.Rémy Coutable2017-10-054-20/+42
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Fetch the remote if a PR has been created/updated after the last fetchRémy Coutable2017-10-051-4/+14
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Ensure Github::Import#execute returns false if any errors occuredRémy Coutable2017-10-051-1/+1
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Set `ref_fetched` to `true` in Github::Import#fetch_pull_requestsRémy Coutable2017-10-051-0/+1
| | | | | | | Since we fetch all the refs we need at the beginning of the import process, we can safely set this attribute. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Don't even check if the branch exists locally as we only use its name in GH ↵Rémy Coutable2017-10-052-56/+4
| | | | | | | | | | | | | import The benefit is that we don't even have to create temp source/target branches. Also, when the source branch of the imported MR is from a fork, name it "user:branch" to mimic how we display it for MR when forks actually exists. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Don't check for commit existence in Github::Representation::Branch#exists?Rémy Coutable2017-10-052-9/+5
| | | | | | | | We don't need to check if the existing branch contains a specific SHA since we only use the branch's name, the SHA is taken from the pull request raw `sha` data. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Keep only the changes that are known to work wellRémy Coutable2017-10-052-23/+28
| | | | | | | | | | | | Also, improved a bit the method names in Github::Representation::PullRequest. Last but not least, ensure temp branch names doesn't contain a `/` as this would create the ref in a subfolder in `refs/heads` (e.g. `refs/heads/gh-123/456/rymai/foo`), and would leave empty directories upon branch deletion (e.g. `refs/heads/gh-123/456/rymai/`. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Fetch and map refs/pull to refs/merge-requests in the GH importJames Lopez2017-10-052-28/+11
| | | | update MR diff
* Fix notes type created from importLin Jen-Shin (godfat)2017-09-295-37/+55
|
* 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>
* Migrate creating/deleting a branch to Gitalyfeature/migrate-branch-operations-to-gitalyAhmad Sherif2017-09-061-1/+1
|
* Decrease ABC threshold to 55.25Maxim Rydkin2017-08-311-40/+42
|
* Fix error when importing a GitHub-wiki repositorybvl-improve-bare-project-importBob Van Landuyt2017-08-231-1/+1
| | | | | This would occur when Wiki's are enabled on GitHub, but there is no wiki repository
* Change `project.path_with_namespace` to `project.disk_path`Douglas Barbosa Alexandre2017-08-071-1/+1
|
* Allow users to disable SSL verification if not connecting to github.comDouglas Barbosa Alexandre2017-08-071-4/+15
|
* Fix small typo on Github::Import#fetch_releasesDouglas Barbosa Alexandre2017-08-071-1/+1
|
* Import GitHub releasesDouglas Barbosa Alexandre2017-08-071-0/+2
|
* Fix Github::Import cache for user idsDouglas Barbosa Alexandre2017-08-071-1/+1
|
* Extrac common timeout to a constant on Github::ClientDouglas Barbosa Alexandre2017-08-071-2/+4
|
* Fix small typoe on Github::Client#root_endpointDouglas Barbosa Alexandre2017-08-071-1/+1
|
* Expire content cache when repository fetch failsDouglas Barbosa Alexandre2017-08-071-2/+3
|
* Set the GitHub API token on options hashDouglas Barbosa Alexandre2017-08-071-2/+2
|
* Move GitHub root endpoint methods to Github::ClientDouglas Barbosa Alexandre2017-08-072-17/+19
|
* Use a custom root endpoint if defined on GH ominiauth provider settingsDouglas Barbosa Alexandre2017-08-071-1/+16
|
* Use project.import_url to fetch repositories from GithubDouglas Barbosa Alexandre2017-08-071-9/+10
|
* Rename many path_with_namespace -> full_pathGabriel Mazetto2017-08-011-1/+1
|
* Add a comment explaining how the branch clean up happensDouglas Barbosa Alexandre2017-06-211-0/+6
|
* Fix Github::Representation::PullRequest#source_branch_exists?Douglas Barbosa Alexandre2017-06-211-1/+1
|
* Fix GitHub importer performance on branch existence checkDouglas Barbosa Alexandre2017-06-213-35/+57
|
* Github - Fix token interpolation when cloning wiki repositoryDouglas Barbosa Alexandre2017-06-121-1/+1
|
* Reset create callbacks for Issues/MRs while importing GitHub projectsDouglas Barbosa Alexandre2017-05-111-0/+2
|
* Fix token interpolation when setting the Github remoteDouglas Barbosa Alexandre2017-05-111-1/+2
|
* [Multiple issue assignee] Fix a number of specsValery Sizov2017-05-051-1/+1
|
* Fix a failing Rubocop rule on masterDouglas Barbosa Alexandre2017-04-261-17/+21
|
* Add basic progress output to GitHub importDouglas Barbosa Alexandre2017-04-251-1/+10
|
* Fix Rubocop offensesDouglas Barbosa Alexandre2017-04-251-2/+6
|
* Fix import of notes on Pull Request diffDouglas Barbosa Alexandre2017-04-253-9/+12
|
* Create project repository only when it not existsDouglas Barbosa Alexandre2017-04-251-1/+1
|
View Lorry Controller Status