Skip to content

Conversation

@rom14514
Copy link
Collaborator

@rom14514 rom14514 commented Nov 10, 2025

Related to issue: #6731

Description:

Add test fixtures dependency

Summary of changes:

  • Add test fixtures dependency

Pre-merge Checklist

@rom14514 rom14514 self-assigned this Nov 10, 2025
@rom14514 rom14514 changed the base branch from main to v.next November 10, 2025 12:38
@rom14514
Copy link
Collaborator Author

rom14514 commented Nov 10, 2025

Note: I have not run the vtest yet because I’m waiting for the daily build that includes the introduction of testFixtures (this PR) to be built. Without it, toolkit will not have access to testFixtures.

Edited: No longer relevant — vtest has been run with both PRs applied and all tests passed successfully.

Copy link
Collaborator

@eri9000 eri9000 left a comment

Choose a reason for hiding this comment

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

👍🏼

@puneet-pdx puneet-pdx self-requested a review November 17, 2025 17:06
Copy link
Collaborator

@puneet-pdx puneet-pdx left a comment

Choose a reason for hiding this comment

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

@rom14514 LGTM 👍🏽

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.

4 participants