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

fix: farm hmr error #11

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

fix: farm hmr error #11

wants to merge 6 commits into from

Conversation

ErKeLost
Copy link

@ErKeLost ErKeLost commented Jul 4, 2024

@sorrycc hi , Farm api and hmr usage issues are solved in the new version, now you can run the full benchmark.

Again, farm enables caching by default when starting the server or build, and the time it takes to write files is huge, so according to mako benchmark project, we need to close caching in the farm test. The time to write to the server is about 40% of the project startup time. I hope you can retest this benchmark. ,grateful

In addition, I think there is a problem with all existing benchmark tests (this also exists in the farm benchmark test. We will summarize it after adding mako test in the future), which is the different target time and backup size. I think we should establish a standard in the future. For specific enviroment (browser or node), We may add various polyfills according to different products. we can establish a unified standard based on https://github.com/browserslist/browserslist and conduct benchmark tests for specific environments.

#1

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.

None yet

1 participant