Skip to content

Daily scrumban status is missing PR assignees#18

Merged
justin-sadakhom merged 2 commits intodevelopfrom
19591-daily-scrumban-status-is-missing-pr-assignees
Apr 25, 2025
Merged

Daily scrumban status is missing PR assignees#18
justin-sadakhom merged 2 commits intodevelopfrom
19591-daily-scrumban-status-is-missing-pr-assignees

Conversation

@justin-sadakhom
Copy link
Copy Markdown
Member

No description provided.

closedAt
# No need to get all pages as issues don't realistically have >100
connectedPrs(first: 100) {
connectedPrs(first: 5) {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ZenHub enforces a complexity limit on graph queries sent to them. I was hitting their limit before reducing this number. We've never seen more than 5 PRs on an issue, so I'm comfortable setting it to 5.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

changes in this file simplify getIssuesByPipeline

@caustin-ziro caustin-ziro self-assigned this Apr 23, 2025
@caustin-ziro caustin-ziro self-requested a review April 23, 2025 14:36
@justin-sadakhom justin-sadakhom merged commit c56274a into develop Apr 25, 2025
@justin-sadakhom justin-sadakhom deleted the 19591-daily-scrumban-status-is-missing-pr-assignees branch April 25, 2025 13:10
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.

2 participants