reactivate ATS. Also upload per-test coverage to staging #17
giovanni-guidinifrontend.yml
on: push
Annotations
1 error and 10 warnings
|
typescript and lint
[@octokit/auth-app] appId option is required
|
|
typescript and lint:
static/app/components/editableText.tsx#L93
React Hook useCallback has missing dependencies: 'displayStatusMessage', 'isEmpty', and 'value'. Either include them or remove the dependency array
|
|
typescript and lint:
static/app/components/editableText.tsx#L99
React Hook useCallback has a missing dependency: 'revertValueAndCloseEditor'. Either include it or remove the dependency array
|
|
typescript and lint:
static/app/components/editableText.tsx#L103
React Hook useEffect has a missing dependency: 'revertValueAndCloseEditor'. Either include it or remove the dependency array
|
|
typescript and lint:
static/app/components/events/rrwebReplayer/index.tsx#L49
React Hook useEffect has a missing dependency: 'loadEvents'. Either include it or remove the dependency array
|
|
typescript and lint:
static/app/components/forms/controls/selectControl.tsx#L346
React Hook useMemo has a missing dependency: 'isInsideModal'. Either include it or remove the dependency array
|
|
typescript and lint:
static/app/components/projects/appStoreConnectContext.tsx#L69
React Hook useEffect has a missing dependency: 'projectDetails'. Either include it or remove the dependency array
|
|
typescript and lint:
static/app/utils/useMemoWithPrevious.spec.tsx#L24
React Hook useMemoWithPrevious has a missing dependency: 'fact'. Either include it or remove the dependency array
|
|
typescript and lint:
static/app/utils/useProjects.tsx#L276
React Hook useEffect has missing dependencies: 'loadProjectsBySlug' and 'shouldLoadSlugs'. Either include them or remove the dependency array. Mutable values like 'slugsRef.current' aren't valid dependencies because mutating them doesn't re-render the component
|
|
typescript and lint:
static/app/utils/useProjects.tsx#L291
React Hook useEffect has missing dependencies: 'shouldLoadSlugs' and 'state.initiallyLoaded'. Either include them or remove the dependency array
|
|
typescript and lint:
static/app/views/performance/landing/widgets/components/queryHandler.tsx#L59
React Hook useEffect has missing dependencies: 'props' and 'query.queryKey'. Either include them or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useEffect call and refer to those specific props inside useEffect
|