From e89f5af7cc39dd976891584ff5b59b64166cbb4f Mon Sep 17 00:00:00 2001 From: Rand Thacker Date: Thu, 30 Apr 2015 17:26:00 -0500 Subject: [PATCH 1/7] Adding update and delete methods --- lib/api/repos/releases.js | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/lib/api/repos/releases.js b/lib/api/repos/releases.js index 029ba09..33c31ee 100644 --- a/lib/api/repos/releases.js +++ b/lib/api/repos/releases.js @@ -11,7 +11,9 @@ function Releases(api) //the api url endpoints this.urls = { releases : '/repos/%s/%s/releases', - latest : '/repos/%s/%s/releases/latest' + info : '/repos/%s/%s/releases/%s', + delete : '/repos/%s/%s/releases/%s', + update : '/repos/%s/%s/releases/%s' } } @@ -21,9 +23,9 @@ Releases.prototype.list = function(user, repo, callback) this.api.get(url, null, callback); }; -Releases.prototype.latest = function(user, repo, callback) +Releases.prototype.info = function(user, repo, tagname, callback) { - var url = util.format(this.urls.latest, user, repo); + var url = util.format(this.urls.info, user, repo, tagname); this.api.get(url, null, callback); }; @@ -33,4 +35,16 @@ Releases.prototype.create = function(user, repo, data, callback) this.api.post(url, data, callback); }; +Releases.prototype.update = function(user, repo, tagname, data, callback) +{ + var url = util.format(this.urls.update, user, repo, tagname); + this.api.patch(url, data, callback); +}; + +Releases.prototype.delete = function(user, repo, tagname, callback) +{ + var url = util.format(this.urls.delete, user, repo, tagname); + this.api.delete(url, null, callback); +}; + module.exports = Releases; \ No newline at end of file From ed21469f1c24f2f42145ae53dd77096aa9201603 Mon Sep 17 00:00:00 2001 From: Rand Thacker Date: Thu, 7 May 2015 16:23:21 -0500 Subject: [PATCH 2/7] Adding filters to repos list --- lib/api/repos/releases.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/api/repos/releases.js b/lib/api/repos/releases.js index 33c31ee..948c816 100644 --- a/lib/api/repos/releases.js +++ b/lib/api/repos/releases.js @@ -17,10 +17,10 @@ function Releases(api) } } -Releases.prototype.list = function(user, repo, callback) +Releases.prototype.list = function(user, repo, filters, callback) { var url = util.format(this.urls.releases, user, repo); - this.api.get(url, null, callback); + this.api.get(url, filters, callback); }; Releases.prototype.info = function(user, repo, tagname, callback) From 30dfffaf79779b4b6063cbcfbaba6b755957023c Mon Sep 17 00:00:00 2001 From: Rand Thacker Date: Tue, 12 May 2015 14:50:17 -0500 Subject: [PATCH 3/7] Adding the merge data for pull merge --- lib/api/pulls.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/api/pulls.js b/lib/api/pulls.js index 64516f6..f38e48b 100644 --- a/lib/api/pulls.js +++ b/lib/api/pulls.js @@ -141,7 +141,7 @@ Pulls.prototype.isMerged = function(user, repo, number, callback) this.api.get(url, null, callback); } -Pulls.prototype.merge = function(user, repo, number, callback) +Pulls.prototype.merge = function(user, repo, number, data, callback) { if(typeof number == 'function'){ callback = number; @@ -156,7 +156,7 @@ Pulls.prototype.merge = function(user, repo, number, callback) } } var url = util.format(this.urls.merge, user, repo, number); - this.api.put(url, null, callback); + this.api.put(url, data, callback); } module.exports = Pulls; \ No newline at end of file From 904eef181eded667ea52cc98a06ce0954be8bc97 Mon Sep 17 00:00:00 2001 From: Rand Thacker Date: Tue, 14 Jul 2015 12:08:22 -0500 Subject: [PATCH 4/7] PR page limit up to 100 instead of default 30 --- lib/api/pulls.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/api/pulls.js b/lib/api/pulls.js index f38e48b..76f535c 100644 --- a/lib/api/pulls.js +++ b/lib/api/pulls.js @@ -7,7 +7,8 @@ function Pulls(api) //the api endpoint urls this.urls = { - pulls : '/repos/%s/%s/pulls', + pulls : '/repos/%s/%s/pulls?per_page=100', + pullCreate : '/repos/%s/%s/pulls', pull : '/repos/%s/%s/pulls/%s', pullCommits : '/repos/%s/%s/pulls/%s/commits', pullFiles : '/repos/%s/%s/pulls/%s/files', @@ -64,7 +65,7 @@ Pulls.prototype.create = function(user, repo, data, callback) user = user[0]; } } - var url = util.format(this.urls.pulls, user, repo); + var url = util.format(this.urls.pullCreate, user, repo); this.api.post(url, data, callback); } From 5f6dbf75dde598a4c84514e2cdbb1948f3d3612e Mon Sep 17 00:00:00 2001 From: Rand Thacker Date: Tue, 14 Jul 2015 12:19:27 -0500 Subject: [PATCH 5/7] Issues page limit up to 100 instead of default 30 --- lib/api/issues.js | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/lib/api/issues.js b/lib/api/issues.js index 348c4c3..335514c 100644 --- a/lib/api/issues.js +++ b/lib/api/issues.js @@ -12,9 +12,10 @@ function Issues(api) //the api url endpoints this.urls = { - issues : '/issues', - repoIssues : '/repos/%s/%s/issues', - issue : '/repos/%s/%s/issues/%s' + issues : '/issues', + repoIssues : '/repos/%s/%s/issues', + repoIssuesPage : '/repos/%s/%s/issues?per_page=100', + issue : '/repos/%s/%s/issues/%s' } } @@ -35,7 +36,7 @@ Issues.prototype.list = function(user, repo, filters, callback) repo = user[1]; user = user[0]; } - var url = util.format(this.urls.repoIssues, user, repo); + var url = util.format(this.urls.repoIssuesPage, user, repo); } else if(typeof filters == 'function') { callback = filters; if(typeof repo == 'object' && !repo.full_name){ @@ -49,9 +50,9 @@ Issues.prototype.list = function(user, repo, filters, callback) repo = user[1]; user = user[0]; } - var url = util.format(this.urls.repoIssues, user, repo); + var url = util.format(this.urls.repoIssuesPage, user, repo); } else { - var url = util.format(this.urls.repoIssues, user, repo); + var url = util.format(this.urls.repoIssuesPage, user, repo); } this.api.get(url, filters, callback); From d541ed77b83eb17c7455f7d3877ad9289feb6fd5 Mon Sep 17 00:00:00 2001 From: Rand Thacker Date: Tue, 14 Jul 2015 12:45:45 -0500 Subject: [PATCH 6/7] Issues page limit up to 100 instead of default 30 --- lib/api/issues.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/api/issues.js b/lib/api/issues.js index 335514c..1b4fa94 100644 --- a/lib/api/issues.js +++ b/lib/api/issues.js @@ -14,12 +14,12 @@ function Issues(api) this.urls = { issues : '/issues', repoIssues : '/repos/%s/%s/issues', - repoIssuesPage : '/repos/%s/%s/issues?per_page=100', + repoIssuesPage : '/repos/%s/%s/issues', issue : '/repos/%s/%s/issues/%s' } } -Issues.prototype.list = function(user, repo, filters, callback) +Issues.prototype.list = function(user, repo, filters, qty, callback) { if(typeof user == 'function'){ var url = this.urls.isses; From 7cba486a2b586de034747761cd0567c4a425097a Mon Sep 17 00:00:00 2001 From: Rand Thacker Date: Tue, 14 Jul 2015 13:07:50 -0500 Subject: [PATCH 7/7] Issues page limit up to 100 instead of default 30 --- lib/api/issues.js | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/lib/api/issues.js b/lib/api/issues.js index 1b4fa94..348c4c3 100644 --- a/lib/api/issues.js +++ b/lib/api/issues.js @@ -12,14 +12,13 @@ function Issues(api) //the api url endpoints this.urls = { - issues : '/issues', - repoIssues : '/repos/%s/%s/issues', - repoIssuesPage : '/repos/%s/%s/issues', - issue : '/repos/%s/%s/issues/%s' + issues : '/issues', + repoIssues : '/repos/%s/%s/issues', + issue : '/repos/%s/%s/issues/%s' } } -Issues.prototype.list = function(user, repo, filters, qty, callback) +Issues.prototype.list = function(user, repo, filters, callback) { if(typeof user == 'function'){ var url = this.urls.isses; @@ -36,7 +35,7 @@ Issues.prototype.list = function(user, repo, filters, qty, callback) repo = user[1]; user = user[0]; } - var url = util.format(this.urls.repoIssuesPage, user, repo); + var url = util.format(this.urls.repoIssues, user, repo); } else if(typeof filters == 'function') { callback = filters; if(typeof repo == 'object' && !repo.full_name){ @@ -50,9 +49,9 @@ Issues.prototype.list = function(user, repo, filters, qty, callback) repo = user[1]; user = user[0]; } - var url = util.format(this.urls.repoIssuesPage, user, repo); + var url = util.format(this.urls.repoIssues, user, repo); } else { - var url = util.format(this.urls.repoIssuesPage, user, repo); + var url = util.format(this.urls.repoIssues, user, repo); } this.api.get(url, filters, callback);