Skip to content

Commit 00d7b99

Browse files
mpadgempadge
authored
Merge pull request #30 from ropensci-review-tools/releases
releases data for ##23
2 parents db6f8c9 + eb7739f commit 00d7b99

File tree

6 files changed

+436
-2
lines changed

6 files changed

+436
-2
lines changed

‎DESCRIPTION

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
Package: repometrics
22
Title: Metrics for Your Code Repository
3-
Version: 0.1.1.061
3+
Version: 0.1.1.065
44
Authors@R:
55
person("Mark", "Padgham", , "mark.padgham@email.com", role = c("aut", "cre"),
66
comment = c(ORCID = "0000-0003-2172-5265"))

‎R/cm-data-gh-releases.R

+43
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
releases_from_gh_api <- function (path, n_per_page = 100L) {
2+
3+
is_test_env <- Sys.getenv ("REPOMETRICS_TESTS") == "true"
4+
5+
u_endpoint <- gh_rest_api_endpoint (path = path, endpoint = "releases")
6+
7+
req <- httr2::request (u_endpoint) |>
8+
httr2::req_url_query (per_page = n_per_page)
9+
10+
body <- NULL
11+
next_page <- 1
12+
13+
while (!is.null (next_page)) {
14+
15+
req <- add_gh_token_to_req (req)
16+
resp <- httr2::req_perform (req)
17+
httr2::resp_check_status (resp)
18+
19+
body <- c (body, httr2::resp_body_json (resp))
20+
21+
next_page <- gh_next_page (resp)
22+
if (is_test_env) {
23+
next_page <- NULL
24+
}
25+
26+
req <- httr2::request (u_endpoint) |>
27+
httr2::req_url_query (per_page = n_per_page) |>
28+
httr2::req_url_query (page = next_page)
29+
}
30+
31+
data.frame (
32+
id = vapply (body, function (i) i$id, integer (1L)),
33+
author_login = vapply (body, function (i) i$author$login, character (1L)),
34+
author_id = vapply (body, function (i) i$author$id, integer (1L)),
35+
tag_name = vapply (body, function (i) i$tag_name, character (1L)),
36+
target_commitish = vapply (body, function (i) i$target_commitish, character (1L)),
37+
name = vapply (body, function (i) i$name, character (1L)),
38+
draft = vapply (body, function (i) i$draft, logical (1L)),
39+
prerelease = vapply (body, function (i) i$prerelease, logical (1L)),
40+
created_at = vapply (body, function (i) i$created_at, character (1L)),
41+
published_at = vapply (body, function (i) i$published_at, character (1L))
42+
)
43+
}

‎codemeta.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
"codeRepository": "https://github.com/ropensci-review-tools/repometrics",
99
"issueTracker": "https://github.com/ropensci-review-tools/repometrics/issues",
1010
"license": "https://spdx.org/licenses/GPL-3.0",
11-
"version": "0.1.1.061",
11+
"version": "0.1.1.065",
1212
"programmingLanguage": {
1313
"@type": "ComputerLanguage",
1414
"name": "R",

‎tests/testthat/gh_api_prs/api.github.com/graphql-5ab71e-POST.json

+256
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,256 @@
1+
{
2+
"data": {
3+
"repository": {
4+
"pullRequests": {
5+
"pageInfo": {
6+
"hasNextPage": true,
7+
"endCursor": "Y3Vyc29yOnYyOpHOBbKXmg=="
8+
},
9+
"nodes": [
10+
{
11+
"number": 19,
12+
"author": {
13+
"login": "jsta"
14+
},
15+
"state": "MERGED",
16+
"closed": true,
17+
"title": "fix warning typos",
18+
"reviewDecision": null,
19+
"reviews": {
20+
"nodes": [
21+
22+
]
23+
},
24+
"merged": true,
25+
"mergedBy": {
26+
"login": "gaborcsardi"
27+
},
28+
"mergeCommit": {
29+
"oid": "f786c10114644afae18c1eb51e662e149451d757"
30+
},
31+
"assignees": {
32+
"nodes": [
33+
34+
]
35+
},
36+
"createdAt": "2016-07-28T17:17:36Z",
37+
"closedAt": "2016-07-28T18:19:24Z",
38+
"updatedAt": "2016-07-28T18:19:28Z",
39+
"closingIssuesReferences": {
40+
"nodes": [
41+
42+
]
43+
},
44+
"commits": {
45+
"nodes": [
46+
{
47+
"commit": {
48+
"oid": "c4c8e971b47953779d1302cfdcf3719baac33eb3"
49+
}
50+
},
51+
{
52+
"commit": {
53+
"oid": "e1081bb7e5c6a9a8d09b76dfba594522a79d3060"
54+
}
55+
}
56+
]
57+
},
58+
"additions": 2,
59+
"changedFiles": 2,
60+
"deletions": 2,
61+
"participants": {
62+
"nodes": [
63+
{
64+
"login": "jsta"
65+
},
66+
{
67+
"login": "codecov-io"
68+
},
69+
{
70+
"login": "gaborcsardi"
71+
}
72+
]
73+
},
74+
"labels": {
75+
"nodes": [
76+
77+
]
78+
},
79+
"body": "",
80+
"totalCommentsCount": 4,
81+
"comments": {
82+
"nodes": [
83+
{
84+
"createdAt": "2016-07-28T17:27:01Z",
85+
"author": {
86+
"login": "codecov-io"
87+
},
88+
"body": "## [Current coverage](https://codecov.io/gh/MangoTheCat/goodpractice/pull/19?src=pr) is 18.61% (diff: 100%)\n\n> Merging [#19](https://codecov.io/gh/MangoTheCat/goodpractice/pull/19?src=pr) into [master](https://codecov.io/gh/MangoTheCat/goodpractice/branch/master?src=pr) will not change coverage\n\n``` diff\n@@ master #19 diff @@\n==========================================\n Files 9 9 \n Lines 274 274 \n Methods 0 0 \n Messages 0 0 \n Branches 0 0 \n==========================================\n Hits 51 51 \n Misses 223 223 \n Partials 0 0 \n```\n\n> Powered by [Codecov](https://codecov.io?src=pr). Last update [cae5198...c064cf7](https://codecov.io/gh/MangoTheCat/goodpractice/compare/cae5198f942621d43cf81b3c0183f6b0fad25323...c064cf713759526a2c09ae1c0505a98f0635867f?src=pr)\n"
89+
},
90+
{
91+
"createdAt": "2016-07-28T18:19:28Z",
92+
"author": {
93+
"login": "gaborcsardi"
94+
},
95+
"body": "Thanks!\n"
96+
}
97+
]
98+
}
99+
},
100+
{
101+
"number": 54,
102+
"author": {
103+
"login": "LiNk-NY"
104+
},
105+
"state": "MERGED",
106+
"closed": true,
107+
"title": "create goodpractice::goodpractice alias from goodpractice::gp, resolves #52",
108+
"reviewDecision": null,
109+
"reviews": {
110+
"nodes": [
111+
{
112+
"author": {
113+
"login": "gaborcsardi"
114+
},
115+
"submittedAt": "2016-12-01T10:41:46Z",
116+
"state": "COMMENTED",
117+
"body": ""
118+
},
119+
{
120+
"author": {
121+
"login": "LiNk-NY"
122+
},
123+
"submittedAt": "2016-12-23T18:51:38Z",
124+
"state": "COMMENTED",
125+
"body": ""
126+
},
127+
{
128+
"author": {
129+
"login": "LiNk-NY"
130+
},
131+
"submittedAt": "2016-12-23T18:52:02Z",
132+
"state": "COMMENTED",
133+
"body": ""
134+
},
135+
{
136+
"author": {
137+
"login": "LiNk-NY"
138+
},
139+
"submittedAt": "2016-12-23T18:55:59Z",
140+
"state": "COMMENTED",
141+
"body": ""
142+
}
143+
]
144+
},
145+
"merged": true,
146+
"mergedBy": {
147+
"login": "gaborcsardi"
148+
},
149+
"mergeCommit": {
150+
"oid": "e6ea88fd81e680de280b01056c0dadcee59dec9f"
151+
},
152+
"assignees": {
153+
"nodes": [
154+
155+
]
156+
},
157+
"createdAt": "2016-11-28T22:35:30Z",
158+
"closedAt": "2017-01-22T19:14:39Z",
159+
"updatedAt": "2017-01-22T19:14:43Z",
160+
"closingIssuesReferences": {
161+
"nodes": [
162+
163+
]
164+
},
165+
"commits": {
166+
"nodes": [
167+
{
168+
"commit": {
169+
"oid": "5ecd642cdc2264f93bed99ec486f70870da33997"
170+
}
171+
},
172+
{
173+
"commit": {
174+
"oid": "4d7b3f3f24da3ba24ed50d07d864801f4466d678"
175+
}
176+
},
177+
{
178+
"commit": {
179+
"oid": "d61adf23e6db4426a66bdf9a7b162b3af3b18837"
180+
}
181+
},
182+
{
183+
"commit": {
184+
"oid": "22e1bd022834e9edcaea6d571648154f08144893"
185+
}
186+
}
187+
]
188+
},
189+
"additions": 36,
190+
"changedFiles": 7,
191+
"deletions": 21,
192+
"participants": {
193+
"nodes": [
194+
{
195+
"login": "LiNk-NY"
196+
},
197+
{
198+
"login": "codecov-io"
199+
},
200+
{
201+
"login": "gaborcsardi"
202+
}
203+
]
204+
},
205+
"labels": {
206+
"nodes": [
207+
208+
]
209+
},
210+
"body": "Changes include: \r\n\r\n* alias function creation and exporting\r\n* authorship information in R format\r\n* reformatted package documentation\r\n\r\nFeel free to change it at your leisure. \r\n\r\nRegards,\r\nMarcel ",
211+
"totalCommentsCount": 11,
212+
"comments": {
213+
"nodes": [
214+
{
215+
"createdAt": "2016-11-28T22:36:06Z",
216+
"author": {
217+
"login": "LiNk-NY"
218+
},
219+
"body": "@gaborcsardi I ran the check on your package and it wasn't passing. I wasn't sure why. \r\n#### Edit: This only happens in RStudio. It seems like a path issue. \r\nHere is the output: \r\n\r\n```\r\nchecking tests ... ERROR\r\nRunning the tests in ‘tests/testthat.R’ failed.\r\nLast 13 lines of output:\r\n > library(testthat)\r\n > library(goodpractice)\r\n > \r\n > test_check(\"goodpractice\")\r\n 1. Error: checks are not overwritten (@test-integrity.R#11) --------------------\r\n non-character argument\r\n 1: unlist(strsplit(rlines, \"\\\\s+\")) at testthat/test-integrity.R:11\r\n 2: strsplit(rlines, \"\\\\s+\")\r\n \r\n testthat results ================================================================\r\n OK: 4 SKIPPED: 0 FAILED: 1\r\n 1. Error: checks are not overwritten (@test-integrity.R#11) \r\n \r\n Error: testthat unit tests failed\r\n Execution halted\r\nR CMD check results\r\n1 error | 0 warnings | 0 notes\r\n```"
220+
},
221+
{
222+
"createdAt": "2016-12-01T00:05:00Z",
223+
"author": {
224+
"login": "LiNk-NY"
225+
},
226+
"body": "@gaborcsardi Any thoughts on this? Thanks. "
227+
},
228+
{
229+
"createdAt": "2016-12-23T18:57:33Z",
230+
"author": {
231+
"login": "codecov-io"
232+
},
233+
"body": "## [Current coverage](https://codecov.io/gh/MangoTheCat/goodpractice/pull/54?src=pr) is 15.40% (diff: 100%)\n> Merging [#54](https://codecov.io/gh/MangoTheCat/goodpractice/pull/54?src=pr) into [master](https://codecov.io/gh/MangoTheCat/goodpractice/branch/master?src=pr) will not change coverage\n\n```diff\n@@ master #54 diff @@\n==========================================\n Files 10 10 \n Lines 331 331 \n Methods 0 0 \n Messages 0 0 \n Branches 0 0 \n==========================================\n Hits 51 51 \n Misses 280 280 \n Partials 0 0 \n```\n\n> Powered by [Codecov](https://codecov.io?src=pr). Last update [e0ecd4f...22e1bd0](https://codecov.io/gh/MangoTheCat/goodpractice/compare/e0ecd4fd932b6d2fd159652cd15cdbc52f4f5e25...22e1bd022834e9edcaea6d571648154f08144893?src=pr)"
234+
},
235+
{
236+
"createdAt": "2017-01-19T18:35:57Z",
237+
"author": {
238+
"login": "LiNk-NY"
239+
},
240+
"body": "Hi Gabor, @gaborcsardi \r\nAny updates on implementing this? "
241+
},
242+
{
243+
"createdAt": "2017-01-22T19:14:43Z",
244+
"author": {
245+
"login": "gaborcsardi"
246+
},
247+
"body": "Thanks!"
248+
}
249+
]
250+
}
251+
}
252+
]
253+
}
254+
}
255+
}
256+
}

0 commit comments

Comments
 (0)