Skip to content

Commit

Permalink
Fix since_last_release.nu script (nushell#385)
Browse files Browse the repository at this point in the history
Missed update of fetch to `http get`
  • Loading branch information
sholderbach authored Mar 18, 2023
1 parent 00b0039 commit 6ec94a8
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions make_release/since_last_release.nu
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# fetch https://api.github.com/repos/nushell/nushell/pulls?q=is%3Apr+merged%3A%3E%3D2021-04-20+ | select html_url user.login title body
# fetch https://api.github.com/search/issues?q=repo:nushell/nushell+is:pr+is:merged+merged:%3E2021-05-08 | get items | select html_url user.login title body
# http get https://api.github.com/repos/nushell/nushell/pulls?q=is%3Apr+merged%3A%3E%3D2021-04-20+ | select html_url user.login title body
# http get https://api.github.com/search/issues?q=repo:nushell/nushell+is:pr+is:merged+merged:%3E2021-05-08 | get items | select html_url user.login title body
# Repos to monitor

def do-work [] {
Expand Down Expand Up @@ -28,7 +28,7 @@ def do-work [] {
let entries = ($site_table | each { |row|
let query_string = $"($query_prefix)($row.repo)($query_suffix)"
# this is for debugging the rate limit. comment it out if things are working well
# fetch -u $env.GITHUB_USERNAME -p $env.GITHUB_PASSWORD https://api.github.com/rate_limit | get resources | select core.limit core.remaining graphql.limit graphql.remaining integration_manifest.limit integration_manifest.remaining search.limit search.remaining
# http get -u $env.GITHUB_USERNAME -p $env.GITHUB_PASSWORD https://api.github.com/rate_limit | get resources | select core.limit core.remaining graphql.limit graphql.remaining integration_manifest.limit integration_manifest.remaining search.limit search.remaining
let site_json = (http get -u $env.GITHUB_USERNAME -p $env.GITHUB_PASSWORD $query_string | get items | select html_url user.login title)

$"## ($row.site)(char nl)(char nl)"
Expand Down

0 comments on commit 6ec94a8

Please sign in to comment.