Skip to content

better test for #17516#17530

Merged
Jarred-Sumner merged 1 commit intomainfrom
dylan/match-test-update
Feb 22, 2025
Merged

better test for #17516#17530
Jarred-Sumner merged 1 commit intomainfrom
dylan/match-test-update

Conversation

@dylan-conway
Copy link
Copy Markdown
Member

What does this PR do?

Follow up for #17516, changing the test to check each match is an expected path, not just checking the total number of matches.

How did you verify your code works?

test

@robobun
Copy link
Copy Markdown
Collaborator

robobun commented Feb 21, 2025

Updated 3:42 PM PT - Feb 21st, 2025

@dylan-conway, your commit 7408e87 has 2 failures in Build #12032:


🧪   try this PR locally:

bunx bun-pr 17530

@Jarred-Sumner Jarred-Sumner merged commit f6c3b92 into main Feb 22, 2025
66 of 70 checks passed
@Jarred-Sumner Jarred-Sumner deleted the dylan/match-test-update branch February 22, 2025 08:24
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.

3 participants