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

Workerize as much functions as possible #15

Open
winston0410 opened this issue Jan 22, 2021 · 12 comments
Open

Workerize as much functions as possible #15

winston0410 opened this issue Jan 22, 2021 · 12 comments
Labels
enhancement New feature or request

Comments

@winston0410
Copy link
Owner

We should move as much work from main thread to worker thread, in order to reduce the workload of the main thread.

As the usage of worker need to be experimented, I think it will be a good idea to first workerize functions in helper.js.

@winston0410 winston0410 added the enhancement New feature or request label Jan 22, 2021
@winston0410
Copy link
Owner Author

After a bit of experiment, I believe that we need to create two set of bundle, one being totally sync and one being async and utilize worker thread. Considering how async would change the behavior of many frontend framework, I believe this is necessary.

@tanekim88
Copy link

Hi Winston, I appreciate your work, and I really love your project rfs-autopilot. But rfs-autopilot stopped working since the last update. Could you please confirm?

@winston0410
Copy link
Owner Author

Hi @tanekim77 thank you for your appreciation. I haven't updated rfs-autopilot for awhile, and i think it is not working because of PostCSS 8 update. You canctake that project if you want to.

@tanekim88
Copy link

tanekim88 commented Mar 18, 2021

@winston0410 Thanks! I would be glad to work on that project to make it work on PostCss 8

@winston0410
Copy link
Owner Author

Great. I will send you a team invitation tomorrow in github. I can add you as package owner in npm as well

@tanekim88
Copy link

@winston0410 Thanks, sounds great!

@winston0410
Copy link
Owner Author

@tanekim77 sorry for the late reply. I have just invited you to the rfs-autopilot repo and you should be able to push at your will right now.

@tanekim88
Copy link

@winston0410 No worries. Thanks for the invite!

@winston0410
Copy link
Owner Author

@tanekim77 do you need to ownership for that package in npm?

@tanekim88
Copy link

@winston0410 It's up to you. I already fixed the bug and pushed the change.

@winston0410
Copy link
Owner Author

Let me add you as the owner. Is tanekim77 your username in npm?

@tanekim88
Copy link

Yes, it's my username. Thanks!

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

No branches or pull requests

2 participants