Skip to content

Update the wishlist leaderboard script to handle PRs#5256

Merged
rzadp merged 3 commits intomasterfrom
rzadp-patch-1
Aug 7, 2024
Merged

Update the wishlist leaderboard script to handle PRs#5256
rzadp merged 3 commits intomasterfrom
rzadp-patch-1

Conversation

@rzadp
Copy link
Copy Markdown
Contributor

@rzadp rzadp commented Aug 6, 2024

Addresses #5085 (comment)

Luckily, in the rest of the script, github API allows (or forces?) us to read the state of PRs the same way as we read the state of issues, so it works without any more changes.

@rzadp rzadp requested a review from kianenigma August 6, 2024 11:30
@rzadp rzadp requested review from a team as code owners August 6, 2024 11:30
@rzadp rzadp added the R0-no-crate-publish-required The change does not require any crates to be re-published. label Aug 6, 2024
@paritytech-review-bot paritytech-review-bot bot requested a review from a team August 6, 2024 11:46
@rzadp rzadp enabled auto-merge August 6, 2024 11:59
Copy link
Copy Markdown
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@rzadp rzadp added this pull request to the merge queue Aug 7, 2024
Merged via the queue into master with commit 34e64c7 Aug 7, 2024
@rzadp rzadp deleted the rzadp-patch-1 branch August 7, 2024 09:54
ordian added a commit that referenced this pull request Aug 7, 2024
* master:
  Umbrella crate: exclude chain-specific crates (#5173)
  Bring reference_hardware.json inline with machine used for weights (#5196)
  Snowbridge on Westend (#5074)
  Run semver check even when no prdoc (#5189)
  Export more from sc-service (#5250)
  Update the wishlist leaderboard script to handle PRs (#5256)
dharjeezy pushed a commit to dharjeezy/polkadot-sdk that referenced this pull request Aug 28, 2024
Addresses
paritytech#5085 (comment)

Luckily, in the rest of the script, github API allows (or forces?) us to
read the state of PRs the same way as we read the state of issues, so it
works without any more changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

R0-no-crate-publish-required The change does not require any crates to be re-published.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants