Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: time to first response being none caused sort failure #15

Merged
merged 1 commit into from
Jun 9, 2023

Conversation

zkoppert
Copy link
Member

@zkoppert zkoppert commented Jun 9, 2023

This pull request fixes a bug where time to first response being none (meaning there are no comments on an issue or pr) fails in the sort function. This change allows the sort to occur and takes issues/prs where time to first response is None and sorts them to the end of the list.

Signed-off-by: Zack Koppert <zkoppert@github.com>
@zkoppert zkoppert self-assigned this Jun 9, 2023
@zkoppert zkoppert added the bug Something isn't working label Jun 9, 2023
@zkoppert zkoppert marked this pull request as ready for review June 9, 2023 18:00
@zkoppert zkoppert merged commit 4354219 into main Jun 9, 2023
@zkoppert zkoppert deleted the ttfr-is-none branch June 9, 2023 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant