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

Add a waterfall to avoid parallel renders #5

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

tex0l
Copy link
Owner

@tex0l tex0l commented Feb 27, 2024

Ugly workaround for #4

@tex0l tex0l linked an issue Feb 27, 2024 that may be closed by this pull request
@tex0l tex0l changed the title global scope pollution of window Run render in forked process Feb 27, 2024
@tex0l
Copy link
Owner Author

tex0l commented Feb 27, 2024

This does not work with a proper build.. There would need to have yet another trick to keep unbuilt the two files in question in the rollup configuration of the vite configuration of the astro configuration... I think I'm going to drop this idea.

@tex0l tex0l changed the title Run render in forked process Add a waterfall to avoid parallel renders Feb 29, 2024
@tex0l
Copy link
Owner Author

tex0l commented Feb 29, 2024

Instead of forking the process, we'll rely on v0.7.2 (see hikerpig/pintora#261) which cleans up the patches it made in the globals, and waterfall execution so that two runs of render do not collide with each other. We also need to expose the waterfall function so that function which need a clean env can wait for the env to be cleaned.

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.

Global scope pollution of window
1 participant