Skip to content

Onboarding and settings refactor to React #1782

Onboarding and settings refactor to React

Onboarding and settings refactor to React #1782

Triggered via pull request August 26, 2024 04:57
Status Failure
Total duration 1m 0s
Artifacts

lint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

11 errors and 4 warnings
phpcs
Process completed with exit code 2.
eslint: src/js/settings/classifai-admin.js#L7
Replace `select·=>·select(·STORE_NAME·).getSettingsScreen()·` with `(·select·)·=>⏎↹↹select(·STORE_NAME·).getSettingsScreen()⏎↹`
eslint: src/js/settings/components/classifai-onboarding/configuration-status.js#L6
Replace `·isFeatureActive,·getFeature,·getEnabledFeaturesSlugs·` with `⏎↹isFeatureActive,⏎↹getFeature,⏎↹getEnabledFeaturesSlugs,⏎`
eslint: src/js/settings/components/classifai-onboarding/configuration-status.js#L8
'setStep' is defined but never used
eslint: src/js/settings/components/classifai-onboarding/configuration-status.js#L10
Replace `select·=>·select(·STORE_NAME·).getSettings()·` with `(·select·)·=>⏎↹↹select(·STORE_NAME·).getSettings()⏎↹`
eslint: src/js/settings/components/classifai-onboarding/configuration-status.js#L28
Replace `'classifai-setup-title'>{·__(·'Welcome·to·ClassifAI',·'classifai'·)·}` with `"classifai-setup-title">⏎↹↹↹↹{·__(·'Welcome·to·ClassifAI',·'classifai'·)·}⏎↹↹↹`
eslint: src/js/settings/components/classifai-onboarding/configure-features.js#L28
React Hook useEffect has a missing dependency: 'setSaveErrors'. Either include it or remove the dependency array
eslint: src/js/settings/components/classifai-onboarding/configure-features.js#L40
React Hook useEffect has missing dependencies: 'enabledFeatures', 'errors.length', 'featureIndex', and 'setStep'. Either include them or remove the dependency array. If 'setStep' changes too often, find the parent component that defines it and wrap that definition in useCallback
eslint: src/js/settings/components/classifai-onboarding/enable-features.js#L18
React Hook useEffect has missing dependencies: 'setStep' and 'step'. Either include them or remove the dependency array. If 'setStep' changes too often, find the parent component that defines it and wrap that definition in useCallback
eslint: src/js/settings/data/hooks.js#L25
React Hook useEffect has a missing dependency: 'isSaving'. Either include it or remove the dependency array