Skip to content

Conversation

@MuxinFeng
Copy link
Contributor

@vercel
Copy link

vercel bot commented Jun 11, 2023

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

Name Status Preview Comments Updated (UTC)
overflow ❌ Failed (Inspect) Jun 14, 2023 3:03am

const RESPONSIVE = 'responsive' as const;
const INVALIDATE = 'invalidate' as const;

export type ComponentType =
Copy link

@kiner-tang kiner-tang Jun 12, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里删掉了导出,可能会导致有从这里引入的地方报错,需要将迁移 RawItem 中的相关代码在这边也导出一下,如:

export type { ComponentType } from './RawItem'
export { OverflowContext } from './RawItem'

@codecov
Copy link

codecov bot commented Jun 12, 2023

Codecov Report

Merging #36 (993bfd5) into master (99531a8) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 993bfd5 differs from pull request most recent head fac55cc. Consider uploading reports for the commit fac55cc to get more accurate results

@@           Coverage Diff           @@
##           master      #36   +/-   ##
=======================================
  Coverage   98.10%   98.10%           
=======================================
  Files           5        6    +1     
  Lines         211      211           
  Branches       71       72    +1     
=======================================
  Hits          207      207           
  Misses          4        4           
Impacted Files Coverage Δ
src/Item.tsx 100.00% <ø> (ø)
src/Overflow.tsx 99.31% <ø> (-0.01%) ⬇️
src/RawItem.tsx 100.00% <ø> (ø)
src/context.ts 100.00% <100.00%> (ø)

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

src/RawItem.tsx Outdated
| React.FC<any>
| keyof React.ReactHTML;

export const OverflowContext = React.createContext<{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Context 放 RawItem 里太奇怪了,直接单独搞个 context.ts 文件吧

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

好滴

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