diff options
| author | Teddy Wing | 2018-04-18 21:59:42 +0200 | 
|---|---|---|
| committer | Teddy Wing | 2018-04-18 21:59:42 +0200 | 
| commit | c3f749355ecde0f1d79524fa9b00f47e32e1437f (patch) | |
| tree | d4befa8e929fea28ee94d2038674e56c335e19a8 | |
| parent | 66e904022aaf55e66b2348ea1d6a45b0b089b59c (diff) | |
| download | redprine-c3f749355ecde0f1d79524fa9b00f47e32e1437f.tar.bz2 | |
new_pull_requests(): Use custom JSON structure
This function was broken because we were trying to subtract untreated
GitHub JSON from our custom saved JSON hashes. To fix it, run the
newly-fetched GitHub JSON through a version of our data extractor that
for now is just copy-pasted from above.
| -rwxr-xr-x | redprine | 22 | 
1 files changed, 21 insertions, 1 deletions
| @@ -68,7 +68,10 @@ function cache_my_pull_requests () {  function new_pull_requests () {  	local pull_requests=$(fetch_pull_requests) -	local new=$(jq "${pull_requests} - ." "$CACHE_FILE") +	local my_prs=$(extract_data_from_pull_request_json "$pull_requests") +	# echo "$my_prs" + +	local new=$(jq "${my_prs} - ." "$CACHE_FILE")  	echo "$new"  } @@ -76,6 +79,23 @@ function new_pull_requests () {  # jq '[ { "html_url": "https://github.com/owner/repo/pull/487", "number": 487, "user": "teddywing", "ref": "9999-this-is-a-test", "created_at": "2018-04-16T16:17:52Z" } ] - .' pulls-my.json  #=> []  # jq '[ { "html_url": "https://github.com/owner/repo/pull/487", "number": 487, "user": "teddywing", "ref": "9999-this-is-a-test", "created_at": "2018-04-16T16:17:52Z" } ] - . | length' pulls-my.json  #=> 0 +function extract_data_from_pull_request_json () { +	local pull_requests="$1" + +	printf "%s\n" "$pull_requests" | +		jq --compact-output '. | +			map({ +				html_url, +				number, +				user: .user.login, +				ref: .head.ref, +				created_at +			}) | +			map( +				select(.user == "teddywing") +			)' +} +  function extract_redmine_issue_numbers_from_pull_request () {  	local pull_requests="$1" | 
