Skip to content

Conversation

talboren
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Jul 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
keep ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2025 4:00pm

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 10, 2025
@talboren talboren enabled auto-merge (squash) July 10, 2025 15:53
cursor[bot]

This comment was marked as outdated.

@talboren talboren disabled auto-merge July 10, 2025 15:56
shahargl
shahargl previously approved these changes Jul 10, 2025
Copy link
Member

@shahargl shahargl left a comment

Choose a reason for hiding this comment

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

Gtm

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 10, 2025
@talboren
Copy link
Member Author

@cursoragent now?

@talboren
Copy link
Member Author

@cursor now?

@talboren talboren enabled auto-merge (squash) July 10, 2025 16:02
@talboren talboren disabled auto-merge July 10, 2025 16:02
Copy link
Member

@shahargl shahargl left a comment

Choose a reason for hiding this comment

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

Lgtm

@talboren talboren merged commit 4d814ab into main Jul 10, 2025
18 of 20 checks passed
@talboren talboren deleted the 5161-bug-keepmul-with-keeplenresults-when-the-lenght-of-results-is-0 branch July 10, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛 Bug]: keep.mul with keep.len(results) when the lenght of results is 0

2 participants