diff --git a/packages/jaeger-ui/src/components/TraceDiff/TraceDiff.tsx b/packages/jaeger-ui/src/components/TraceDiff/TraceDiff.tsx index bd6e0426ef..ff9c8cbab1 100644 --- a/packages/jaeger-ui/src/components/TraceDiff/TraceDiff.tsx +++ b/packages/jaeger-ui/src/components/TraceDiff/TraceDiff.tsx @@ -14,7 +14,6 @@ import * as React from 'react'; import { History as RouterHistory } from 'history'; -import queryString from 'query-string'; import { connect } from 'react-redux'; import { match } from 'react-router-dom'; import { bindActionCreators, Dispatch } from 'redux'; diff --git a/packages/jaeger-ui/src/components/common/RelativeDate.tsx b/packages/jaeger-ui/src/components/common/RelativeDate.tsx index e72dae7eda..31c44e4cb6 100644 --- a/packages/jaeger-ui/src/components/common/RelativeDate.tsx +++ b/packages/jaeger-ui/src/components/common/RelativeDate.tsx @@ -12,6 +12,7 @@ // See the License for the specific language governing permissions and // limitations under the License. +import React from 'react'; import dayjs from 'dayjs'; import { formatRelativeDate } from '../../utils/date'; @@ -19,15 +20,15 @@ import { formatRelativeDate } from '../../utils/date'; type Props = { fullMonthName: boolean | undefined | null; includeTime: boolean | undefined | null; - value: number | Date | any; + value: number | Date; }; // TODO typescript doesn't understand text or null as react nodes // https://github.com/Microsoft/TypeScript/issues/21699 -export default function RelativeDate(props: Props): any { +export default function RelativeDate(props: Props): JSX.Element { const { value, includeTime, fullMonthName } = props; const m = dayjs.isDayjs(value) ? value : dayjs(value); const dateStr = formatRelativeDate(m, Boolean(fullMonthName)); const timeStr = includeTime ? `, ${m.format('h:mm:ss a')}` : ''; - return `${dateStr}${timeStr}`; + return {`${dateStr}${timeStr}`}; } diff --git a/packages/jaeger-ui/src/utils/DraggableManager/DraggableManager.tsx b/packages/jaeger-ui/src/utils/DraggableManager/DraggableManager.tsx index 090da8bc2e..5667bae7d3 100644 --- a/packages/jaeger-ui/src/utils/DraggableManager/DraggableManager.tsx +++ b/packages/jaeger-ui/src/utils/DraggableManager/DraggableManager.tsx @@ -60,10 +60,10 @@ export default class DraggableManager { tag: string | TNil; // handlers for integration with DOM elements - handleMouseEnter: (event: React.MouseEvent) => void; - handleMouseMove: (event: React.MouseEvent) => void; - handleMouseLeave: (event: React.MouseEvent) => void; - handleMouseDown: (event: React.MouseEvent) => void; + handleMouseEnter: (event: React.MouseEvent) => void; + handleMouseMove: (event: React.MouseEvent) => void; + handleMouseLeave: (event: React.MouseEvent) => void; + handleMouseDown: (event: React.MouseEvent) => void; constructor({ getBounds, tag, resetBoundsOnResize = true, ...rest }: DraggableManagerOptions) { this.handleMouseDown = this._handleDragEvent; @@ -142,7 +142,7 @@ export default class DraggableManager { this._bounds = undefined; }; - _handleMinorMouseEvent = (event: React.MouseEvent) => { + _handleMinorMouseEvent = (event: React.MouseEvent) => { const { button, clientX, type: eventType } = event; if (this._isDragging || button !== LEFT_MOUSE_BUTTON) { return; @@ -175,7 +175,7 @@ export default class DraggableManager { }); }; - _handleDragEvent = (event: MouseEvent | React.MouseEvent) => { + _handleDragEvent = (event: MouseEvent | React.MouseEvent) => { const { button, clientX, type: eventType } = event; let type: EUpdateTypes | null = null; let handler: ((update: DraggingUpdate) => void) | TNil; diff --git a/packages/jaeger-ui/src/utils/DraggableManager/types.tsx b/packages/jaeger-ui/src/utils/DraggableManager/types.tsx index 16ab7948d4..b9775f2b0d 100644 --- a/packages/jaeger-ui/src/utils/DraggableManager/types.tsx +++ b/packages/jaeger-ui/src/utils/DraggableManager/types.tsx @@ -26,7 +26,7 @@ export type DraggableBounds = { }; export type DraggingUpdate = { - event: React.MouseEvent | MouseEvent; + event: React.MouseEvent | MouseEvent; manager: DraggableManager; tag: string | TNil; type: EUpdateTypes; diff --git a/packages/jaeger-ui/src/utils/config/process-deprecation.tsx b/packages/jaeger-ui/src/utils/config/process-deprecation.tsx index 196de73732..daa6c78d15 100644 --- a/packages/jaeger-ui/src/utils/config/process-deprecation.tsx +++ b/packages/jaeger-ui/src/utils/config/process-deprecation.tsx @@ -34,11 +34,7 @@ interface IDeprecation { * - The value at the deprecated config property is moved to the new property * - The value at the deprecated config property is ignored in favor of the value at the new property */ -export default function processDeprecation( - config: Record, - deprecation: IDeprecation, - issueWarning: boolean -) { +export default function processDeprecation(config: object, deprecation: IDeprecation, issueWarning: boolean) { const { formerKey, currentKey } = deprecation; if (_has(config, formerKey)) { let isTransfered = false; diff --git a/packages/jaeger-ui/src/utils/readJsonFile.tsx b/packages/jaeger-ui/src/utils/readJsonFile.tsx index 91753f720c..9f5fcde1d6 100644 --- a/packages/jaeger-ui/src/utils/readJsonFile.tsx +++ b/packages/jaeger-ui/src/utils/readJsonFile.tsx @@ -22,6 +22,7 @@ export default function readJsonFile(fileList: { file: File }) { } try { resolve(JSON.parse(reader.result)); + /* eslint-disable @typescript-eslint/no-explicit-any */ } catch (error: any) { reject(new Error(`Error parsing JSON: ${error.message}`)); } @@ -38,7 +39,6 @@ export default function readJsonFile(fileList: { file: File }) { try { reader.readAsText(fileList.file); } catch (error: any) { - // eslint-disable-next-line no-console reject(new Error(`Error reading the JSON file: ${error.message}`)); } }); diff --git a/packages/jaeger-ui/src/utils/tracking/common.tsx b/packages/jaeger-ui/src/utils/tracking/common.tsx index fa7994c41c..a798f5cf2c 100644 --- a/packages/jaeger-ui/src/utils/tracking/common.tsx +++ b/packages/jaeger-ui/src/utils/tracking/common.tsx @@ -15,6 +15,6 @@ export const OPEN = 'open'; export const CLOSE = 'close'; -export function getToggleValue(value: any) { +export function getToggleValue(value: boolean) { return value ? CLOSE : OPEN; } diff --git a/packages/jaeger-ui/src/utils/tracking/getTrackFilter.tsx b/packages/jaeger-ui/src/utils/tracking/getTrackFilter.tsx index da4f47b930..71509ba4c2 100644 --- a/packages/jaeger-ui/src/utils/tracking/getTrackFilter.tsx +++ b/packages/jaeger-ui/src/utils/tracking/getTrackFilter.tsx @@ -16,6 +16,8 @@ import _throttle from 'lodash/throttle'; import { trackEvent } from '.'; +import { TNil } from '../../types'; + // export for tests export const ACTION_FILTER_SET = 'set'; export const ACTION_FILTER_CLEAR = 'clear'; @@ -33,5 +35,5 @@ const getTrackFilterClear = (category: string) => export default function getTrackFilter(category: string) { const trackFilterSet = getTrackFilterSet(category); const trackFilterClear = getTrackFilterClear(category); - return (value: any) => (value ? trackFilterSet() : trackFilterClear()); + return (value: string | TNil) => (value ? trackFilterSet() : trackFilterClear()); } diff --git a/packages/jaeger-ui/src/utils/ts/pluckTruthy.tsx b/packages/jaeger-ui/src/utils/ts/pluckTruthy.tsx index 1d7515f3d4..a43a8955d4 100644 --- a/packages/jaeger-ui/src/utils/ts/pluckTruthy.tsx +++ b/packages/jaeger-ui/src/utils/ts/pluckTruthy.tsx @@ -12,6 +12,7 @@ // See the License for the specific language governing permissions and // limitations under the License. +/* eslint-disable @typescript-eslint/no-explicit-any */ // Because TypeScript doesn't believe in Array#filter(Boolean) export default function pluckTruthy(values: (T | any)[]): T[] { const rv: T[] = [];