Skip to content

chore: add ci checks, order fields, fix types, and simplify logic #28

chore: add ci checks, order fields, fix types, and simplify logic

chore: add ci checks, order fields, fix types, and simplify logic #28

Triggered via pull request August 31, 2023 13:03
Status Failure
Total duration 36s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

ci.yml

on: pull_request
Lint source files with ESLint
25s
Lint source files with ESLint
Lint source files with Prettier
22s
Lint source files with Prettier
Scan for known vulnerabilities in dependencies with npm audit
25s
Scan for known vulnerabilities in dependencies with npm audit
Check for unused and missing dependencies with depcheck
24s
Check for unused and missing dependencies with depcheck
Check for outdated packages with npm outdated and npm-check-updates
25s
Check for outdated packages with npm outdated and npm-check-updates
Spell Check with Typos
3s
Spell Check with Typos
Check the commit conforms to the Conventional Commits specification with commitlint
13s
Check the commit conforms to the Conventional Commits specification with commitlint
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 11 warnings
Lint source files with ESLint: index.d.ts#L301
File has too many lines (327). Maximum allowed is 300
Lint source files with ESLint: index.js#L2
Expected 'multiple' syntax before 'single' syntax
Lint source files with ESLint: index.js#L12
Missing return type on function
Lint source files with ESLint: index.js#L17
Missing return type on function
Lint source files with ESLint: index.js#L22
Missing return type on function
Lint source files with ESLint: index.js#L27
Missing return type on function
Lint source files with ESLint: index.js#L31
Missing return type on function
Lint source files with ESLint: index.js#L35
Missing return type on function
Spell Check with Typos
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@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 source files with ESLint: src/components/ListEmpty.js#L5
Missing JSDoc comment
Lint source files with ESLint: src/components/Picker.js#L52
Missing JSDoc @returns declaration
Lint source files with ESLint: src/components/Picker.js#L55
Missing JSDoc @param "arrayOne" type
Lint source files with ESLint: src/components/Picker.js#L56
Missing JSDoc @param "arrayTwo" type
Lint source files with ESLint: src/components/Picker.js#L72
Missing JSDoc comment
Lint source files with ESLint: src/components/Picker.js#L241
React Hook useEffect has missing dependencies: 'ITEM_SCHEMA.value', 'items', 'multiple', and 'value'. Either include them or remove the dependency array
Lint source files with ESLint: src/components/Picker.js#L257
React Hook useEffect has missing dependencies: 'closeOnBackPressed' and 'setOpen'. Either include them or remove the dependency array. If 'setOpen' changes too often, find the parent component that defines it and wrap that definition in useCallback
Lint source files with ESLint: src/components/Picker.js#L276
React Hook useEffect has a missing dependency: 'ITEM_SCHEMA.value'. Either include it or remove the dependency array
Lint source files with ESLint: src/components/Picker.js#L332
React Hook useEffect has missing dependencies: 'ITEM_SCHEMA.value', 'multiple', and 'onChangeValue'. Either include them or remove the dependency array. If 'onChangeValue' changes too often, find the parent component that defines it and wrap that definition in useCallback
Lint source files with ESLint: src/components/Picker.js#L353
'scroll' was used before it was defined