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

Allow multiple simultaneous compile attempts by queuing them. #136

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DDR0
Copy link

@DDR0 DDR0 commented Jul 1, 2022

If plugin.js is invoked twice by a misconfigured webpack, say because I haven't figured out how to correctly compile two different web workers yet, print a warning and compile them both anyway instead of returning a mysterious error message.

This is a superset of PR #135, as it does represent a behaviour departure from base instead of a strict error message improvement. 🙂

Say, if you accidentally launched two processes simultaneously...
@DDR0 DDR0 changed the title Allow multiple compiles Allow multiple simultaneous compile attempts by queuing them. Jul 1, 2022
@DDR0 DDR0 force-pushed the allow-multiple-compiles branch from eb0ac9d to 35691c4 Compare July 1, 2022 19:13
@DDR0
Copy link
Author

DDR0 commented Jul 1, 2022

Oh, when you merge this, use the command-line instructions. I messed something up with the web interface and it's missing a try/catch block around the await previousRun statement.

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.

1 participant