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

enhance: clean cache if not exist anymore #132

Merged
merged 2 commits into from
Jul 21, 2023
Merged

enhance: clean cache if not exist anymore #132

merged 2 commits into from
Jul 21, 2023

Conversation

zombieJ
Copy link
Member

@zombieJ zombieJ commented Jul 21, 2023

如果样式已经移除,则清理 cacheMap 以优化页面来回切换时不必要的损耗。

@codecov-commenter
Copy link

codecov-commenter commented Jul 21, 2023

Codecov Report

Patch coverage: 95.45% and project coverage change: -0.02 ⚠️

Comparison is base (523e1f1) 94.82% compared to head (5a98ba9) 94.80%.

❗ Current head 5a98ba9 differs from pull request most recent head 15d8786. Consider uploading reports for the commit 15d8786 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
- Coverage   94.82%   94.80%   -0.02%     
==========================================
  Files          24       24              
  Lines        2029     2041      +12     
  Branches      318      321       +3     
==========================================
+ Hits         1924     1935      +11     
- Misses        105      106       +1     
Impacted Files Coverage Δ
src/hooks/useStyleRegister/cacheMapUtil.ts 95.78% <95.45%> (-0.60%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions
Copy link

github-actions bot commented Jul 21, 2023

😭 Deploy PR Preview 15d8786 failed. Build logs

🤖 By surge-preview

@afc163
Copy link
Member

afc163 commented Jul 21, 2023

是不是应该先发 alpha 版本做测试。。

@zombieJ zombieJ merged commit 7ad8b6f into master Jul 21, 2023
@zombieJ zombieJ deleted the adjust branch July 21, 2023 05:39
@zombieJ
Copy link
Member Author

zombieJ commented Jul 21, 2023

是不是应该先发 alpha 版本做测试。。

嗯,之前主体应该是 alpha 更保险一些。

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.

4 participants