Skip to content

Conversation

@zombieJ
Copy link
Member

@zombieJ zombieJ commented Oct 10, 2022

@vercel
Copy link

vercel bot commented Oct 10, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
tabs ✅ Ready (Inspect) Visit Preview Oct 10, 2022 at 5:15PM (UTC)

@codecov
Copy link

codecov bot commented Oct 10, 2022

Codecov Report

Merging #595 (7e2c9c4) into master (2f4af9b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #595   +/-   ##
=======================================
  Coverage   99.64%   99.65%           
=======================================
  Files          15       17    +2     
  Lines         558      574   +16     
  Branches      134      136    +2     
=======================================
+ Hits          556      572   +16     
  Misses          2        2           
Impacted Files Coverage Δ
src/TabNavList/ExtraContent.tsx 100.00% <100.00%> (ø)
src/TabNavList/index.tsx 99.45% <100.00%> (-0.01%) ⬇️
src/hooks/useVisibleRange.ts 100.00% <100.00%> (ø)
src/util.ts 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 10, 2022

This pull request introduces 1 alert and fixes 1 when merging 49e946a into 2f4af9b - view on LGTM.com

new alerts:

  • 1 for Useless conditional

fixed alerts:

  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 10, 2022

This pull request introduces 1 alert and fixes 1 when merging aa4a0db into 2f4af9b - view on LGTM.com

new alerts:

  • 1 for Useless conditional

fixed alerts:

  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 10, 2022

This pull request introduces 1 alert and fixes 1 when merging d548048 into 2f4af9b - view on LGTM.com

new alerts:

  • 1 for Useless conditional

fixed alerts:

  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 10, 2022

This pull request introduces 1 alert and fixes 1 when merging 5acd2a4 into 2f4af9b - view on LGTM.com

new alerts:

  • 1 for Useless conditional

fixed alerts:

  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 10, 2022

This pull request introduces 1 alert and fixes 1 when merging 543fb29 into 2f4af9b - view on LGTM.com

new alerts:

  • 1 for Useless conditional

fixed alerts:

  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 10, 2022

This pull request introduces 1 alert and fixes 1 when merging 84aa41b into 2f4af9b - view on LGTM.com

new alerts:

  • 1 for Useless conditional

fixed alerts:

  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 10, 2022

This pull request introduces 1 alert and fixes 1 when merging 05c0532 into 2f4af9b - view on LGTM.com

new alerts:

  • 1 for Useless conditional

fixed alerts:

  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 10, 2022

This pull request introduces 1 alert and fixes 1 when merging 228c303 into 2f4af9b - view on LGTM.com

new alerts:

  • 1 for Useless conditional

fixed alerts:

  • 1 for Useless conditional

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 10, 2022

This pull request fixes 1 alert when merging 7e2c9c4 into 2f4af9b - view on LGTM.com

fixed alerts:

  • 1 for Useless conditional

@zombieJ zombieJ marked this pull request as ready for review October 10, 2022 17:17
@zombieJ
Copy link
Member Author

zombieJ commented Oct 10, 2022

改动比较大,要发个 minor

@zombieJ zombieJ merged commit ca8a22d into master Oct 11, 2022
@zombieJ zombieJ deleted the fix-tabs branch October 11, 2022 03:26
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.

3 participants