feat(DragDrop): introduce new package and deprecate old implementation #6249
Triggered via pull request
October 17, 2023 18:11
nicolethoen
synchronize
#8620
Status
Failure
Total duration
12m 0s
Artifacts
1
pr-preview.yml
on: pull_request_target
Annotations
14 errors and 20 warnings
docs
Process completed with exit code 2.
|
docs
Process completed with exit code 1.
|
docs
Process completed with exit code 1.
|
lint:
packages/react-drag-drop/src/components/DragDrop/DragButton.tsx#L17
Replace `·className={css(className,·buttonStyles.button,·buttonStyles.modifiers.plain)}·aria-label="Drag·button"·{...props}` with `⏎····className={css(className,·buttonStyles.button,·buttonStyles.modifiers.plain)}⏎····aria-label="Drag·button"⏎····{...props}⏎··`
|
lint:
packages/react-drag-drop/src/components/DragDrop/DragDropSort.tsx#L70
Replace `item` with `(item)`
|
lint:
packages/react-drag-drop/src/components/DragDrop/DragDropSort.tsx#L72
Replace `item` with `(item)`
|
|
lint:
packages/react-drag-drop/src/components/DragDrop/DragDropSort.tsx#L139
Replace `········` with `··········`
|
|
|
lint:
packages/react-drag-drop/src/components/DragDrop/DragDropSort.tsx#L143
Replace `⏎··········{renderedChildren}⏎········` with `{renderedChildren}`
|
lint:
packages/react-drag-drop/src/components/DragDrop/Draggable.tsx#L39
Replace `⏎········{...attributes}⏎········{...listeners}⏎·····` with `·{...attributes}·{...listeners}`
|
lint:
packages/react-drag-drop/src/components/DragDrop/examples/DataListDraggable.tsx#L6
Replace `idx` with `(idx)`
|
test_jest
Process completed with exit code 1.
|
build
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
docs
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
demo_app
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
lint
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
lint:
packages/react-core/src/components/BackToTop/BackToTop.tsx#L36
The 'toggleVisible' function makes the dependencies of useEffect Hook (at line 74) change on every render. Move it inside the useEffect callback. Alternatively, wrap the definition of 'toggleVisible' in its own useCallback() Hook
|
lint:
packages/react-core/src/components/CalendarMonth/CalendarMonth.tsx#L179
React Hook useEffect has a missing dependency: 'focusedDate'. Either include it or remove the dependency array
|
lint:
packages/react-core/src/components/CalendarMonth/CalendarMonth.tsx#L186
React Hook useEffect has a missing dependency: 'shouldFocus'. Either include it or remove the dependency array
|
lint:
packages/react-core/src/components/DatePicker/DatePicker.tsx#L134
React Hook React.useMemo has a missing dependency: 'placeholder.length'. Either include it or remove the dependency array
|
lint:
packages/react-core/src/components/DatePicker/DatePicker.tsx#L145
React Hook React.useEffect has a missing dependency: 'dateParse'. Either include it or remove the dependency array. If 'dateParse' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
lint:
packages/react-core/src/components/DatePicker/DatePicker.tsx#L156
React Hook React.useEffect has missing dependencies: 'dateIsRequired', 'dateParse', 'emptyDateText', 'errorText', 'pristine', 'setError', and 'textInputFocused'. Either include them or remove the dependency array. If 'dateParse' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
lint:
packages/react-core/src/components/DatePicker/DatePicker.tsx#L222
React Hook useImperativeHandle has an unnecessary dependency: 'selectOpen'. Either exclude it or remove the dependency array
|
lint:
packages/react-core/src/components/Drawer/DrawerPanelContent.tsx#L245
Assignments to the 'setInitialVals' variable from inside React Hook React.useCallback will be lost after each render. To preserve the value over time, store it in a useRef Hook and keep the mutable value in the '.current' property. Otherwise, you can move this variable directly inside React.useCallback
|
lint:
packages/react-core/src/components/Drawer/DrawerPanelContent.tsx#L261
React Hook React.useCallback has missing dependencies: 'handleControlMove' and 'position'. Either include them or remove the dependency array
|
lint:
packages/react-core/src/components/Drawer/DrawerPanelContent.tsx#L262
React Hook React.useCallback has missing dependencies: 'callbackTouchEnd', 'callbackTouchMove', 'currWidth', 'id', and 'onResize'. Either include them or remove the dependency array. If 'onResize' changes too often, find the parent component that defines it and wrap that definition in useCallback
|
test_jest
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
test_integration (3, 5)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2, actions/download-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
test_integration (1, 5)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2, actions/download-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
test_integration (0, 5)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2, actions/download-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
test_integration (4, 5)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2, actions/download-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
test_integration (2, 5)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-node@v1, actions/cache@v2, actions/download-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Artifacts
Produced during runtime
Name | Size | |
---|---|---|
demo-app
Expired
|
5.24 MB |
|