Skip to content

fix(miners): sort pull requests by state-relevant timestamp#187

Closed
kiannidev wants to merge 1 commit into
entrius:testfrom
kiannidev:fix/186-miners-pr-state-sort
Closed

fix(miners): sort pull requests by state-relevant timestamp#187
kiannidev wants to merge 1 commit into
entrius:testfrom
kiannidev:fix/186-miners-pr-state-sort

Conversation

@kiannidev

@kiannidev kiannidev commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Sort miner pull request results by merged_at for MERGED, closed_at for CLOSED, and created_at for OPEN
  • Shared PR_ORDER_BY clause used by both getPullRequests and getPullRequestsByRepo

Related Issues

Fixes #186

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other (describe below)

Testing

  • npm run build in packages/das passes

Checklist

  • I have read the Contributing Guide
  • Code builds without errors
  • New and existing tests pass (if applicable)
  • Documentation updated (if applicable)
  • No unnecessary dependencies added

@xiao-xiao-mao

xiao-xiao-mao Bot commented Jun 15, 2026

Copy link
Copy Markdown

You already have 2 open PRs in this repo. The cap is 2, so closing this one until a maintainer can triage your existing PRs. Please don't open any more — they'll be closed too.

@xiao-xiao-mao xiao-xiao-mao Bot closed this Jun 15, 2026
@kiannidev

Copy link
Copy Markdown
Contributor Author

@xiao-xiao-mao
can this PR be opened after other PRs are closed?

@kiannidev

Copy link
Copy Markdown
Contributor Author

@anderdc
Can you reopen this PR?
This is closed because it was third PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] miners API: pull requests sorted by created_at instead of state-relevant timestamp

1 participant