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

virtualise test entrypoint #1924

Merged
merged 1 commit into from
May 17, 2024
Merged

virtualise test entrypoint #1924

merged 1 commit into from
May 17, 2024

Conversation

mansona
Copy link
Member

@mansona mansona commented May 16, 2024

No description provided.

@mansona mansona added the enhancement New feature or request label May 16, 2024
@mansona mansona force-pushed the virtual-test-entrypoint branch 2 times, most recently from 9f6a400 to 77ee831 Compare May 16, 2024 15:34
Copy link
Contributor

@ef4 ef4 left a comment

Choose a reason for hiding this comment

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

I think this either needs to be radically reduced down to things that only happen in the tests entrypoint or (if we really do need a lot of this code) needs to share the implementation with the app entrypoint. I don't want to just duplicate the implementation.

packages/core/src/virtual-test-entrypoint.ts Outdated Show resolved Hide resolved
packages/core/src/virtual-test-entrypoint.ts Outdated Show resolved Hide resolved
@BlueCutOfficial BlueCutOfficial force-pushed the virtual-test-entrypoint branch from 0ed4657 to 77ee831 Compare May 17, 2024 10:50
@mansona mansona force-pushed the virtual-test-entrypoint branch from 77ee831 to b811064 Compare May 17, 2024 16:34
@mansona mansona force-pushed the virtual-test-entrypoint branch from b811064 to 244ab55 Compare May 17, 2024 18:34
@mansona mansona requested a review from ef4 May 17, 2024 18:39
Copy link
Contributor

@ef4 ef4 left a comment

Choose a reason for hiding this comment

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

Much smaller!

@ef4 ef4 merged commit 6b85eb1 into main May 17, 2024
176 checks passed
@ef4 ef4 deleted the virtual-test-entrypoint branch May 17, 2024 22:16
@github-actions github-actions bot mentioned this pull request May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants