Fix: Breadcrumb and Checkbox require cycles warnings

[fervailanti]

Summary

Some circular requiring were causing this warning to appear:

Changelog

• [General] [Fixed] - Fixed Pressable requiring in Breadcrumb component
• [General] [Fixed] - Fixed Box requiring in Checkbox (SizedIcon) component

Test Plan

The warning disappears.

[vercel]

@fervailanti is attempting to deploy a commit to the Geekyants Team Team on Vercel.

A member of the Team first needs to authorize it.

[JMRBDev]

Will this eventually be merged?

[3kwtloo]

Thx for the PR - I suggest applying these changes as well to src/components/composites/Transitions/Slide.tsx:

diff --git a/node_modules/native-base/src/components/composites/Transitions/Slide.tsx b/node_modules/native-base/src/components/composites/Transitions/Slide.tsx index 7e63779..3e589fa 100644 --- a/node_modules/native-base/src/components/composites/Transitions/Slide.tsx +++ b/node_modules/native-base/src/components/composites/Transitions/Slide.tsx @@ -4,7 +4,7 @@ import { useThemeProps } from '../../../hooks/useThemeProps'; import type { ISlideProps } from './types'; import PresenceTransition from './PresenceTransition'; import { useHasResponsiveProps } from '../../../hooks/useHasResponsiveProps'; -import { Overlay } from '../../primitives'; +import { Overlay } from '../../primitives/Overlay/Overlay';

I was getting another dip cycle with the latest version from npm as well

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.