Fix eslint

src/legacy/core_plugins/data/public/query/query_bar/components/query_bar_top_row.tsx

@@ -78,16 +78,16 @@ function QueryBarTopRowUI(props: Props) {
   const kueryQuerySyntaxLink: string = docLinks!.links.query.kueryQuerySyntax;
 
   const queryLanguage = props.query && props.query.language;
-  let persistedLog: PersistedLog | undefined;
+  const persistedLog = React.useRef<PersistedLog | undefined>(undefined);
 
   useEffect(() => {
     if (!props.query) return;
-    persistedLog = getQueryLog(uiSettings!, store, appName, props.query.language);
+    persistedLog.current = getQueryLog(uiSettings!, store, appName, props.query.language);
   }, [queryLanguage]);
 
   function onClickSubmitButton(event: React.MouseEvent<HTMLButtonElement>) {
-    if (persistedLog && props.query) {
-      persistedLog.add(props.query.query);
+    if (persistedLog.current && props.query) {
+      persistedLog.current.add(props.query.query);
     }
     event.preventDefault();
     onSubmit({ query: props.query, dateRange: getDateRange() });
@@ -184,7 +184,7 @@ function QueryBarTopRowUI(props: Props) {
           screenTitle={props.screenTitle}
           onChange={onQueryChange}
           onSubmit={onInputSubmit}
-          persistedLog={persistedLog}
+          persistedLog={persistedLog.current}
         />
       </EuiFlexItem>
     );

src/legacy/core_plugins/data/public/search/search_bar/components/saved_query_management/save_query_form.tsx

@@ -83,7 +83,7 @@ export const SaveQueryForm: FunctionComponent<Props> = ({
       setSavedQueries(sortedAllSavedQueries);
     };
     fetchQueries();
-  }, []);
+  }, [savedQueryService]);
 
   const savedQueryDescriptionText = i18n.translate(
     'data.search.searchBar.savedQueryDescriptionText',

src/legacy/core_plugins/data/public/search/search_bar/components/saved_query_management/saved_query_management_component.tsx

@@ -75,7 +75,7 @@ export const SavedQueryManagementComponent: FunctionComponent<Props> = ({
     if (isOpen) {
       fetchCountAndSavedQueries();
     }
-  }, [isOpen, activePage]);
+  }, [isOpen, activePage, savedQueryService]);
 
   const goToPage = (pageNumber: number) => {
     setActivePage(pageNumber);

src/legacy/core_plugins/data/public/search/search_bar/components/search_bar.test.tsx

@@ -33,14 +33,14 @@ const timefilterSetupMock = timefilterServiceMock.createSetupContract();
 
 jest.mock('../../../../../data/public', () => {
   return {
-    FilterBar: () => <div className="filterBar"></div>,
-    QueryBarInput: () => <div className="queryBar"></div>,
+    FilterBar: () => <div className="filterBar" />,
+    QueryBarInput: () => <div className="queryBar" />,
   };
 });
 
 jest.mock('../../../query/query_bar', () => {
   return {
-    QueryBarTopRow: () => <div className="queryBar"></div>,
+    QueryBarTopRow: () => <div className="queryBar" />,
   };
 });
 

src/legacy/core_plugins/data/public/search/search_bar/components/search_bar.tsx

@@ -368,7 +368,7 @@ class SearchBarUI extends Component<SearchBarProps, State> {
         onLoad={this.onLoadSavedQuery}
         savedQueryService={this.savedQueryService}
         onClearSavedQuery={this.props.onClearSavedQuery}
-      ></SavedQueryManagementComponent>
+      />
     );
 
     let queryBar;

src/legacy/core_plugins/expressions/public/np_ready/public/expression_renderer.tsx

@@ -57,6 +57,7 @@ export const ExpressionRendererImplementation = ({
   const [state, setState] = useState<State>({ ...defaultState });
 
   // Re-fetch data automatically when the inputs change
+  /* eslint-disable react-hooks/exhaustive-deps */
   useEffect(() => {
     if (handlerRef.current) {
       handlerRef.current.update(expression, options);
@@ -68,6 +69,7 @@ export const ExpressionRendererImplementation = ({
     options.variables,
     options.disableCaching,
   ]);
+  /* eslint-enable react-hooks/exhaustive-deps */
 
   // Initialize the loader only once
   useEffect(() => {

src/legacy/core_plugins/navigation/public/top_nav_menu/top_nav_menu.test.tsx

@@ -29,7 +29,7 @@ jest.mock('ui/new_platform');
 
 const dataShim = {
   ui: {
-    SearchBar: () => <div className="searchBar"></div>,
+    SearchBar: () => <div className="searchBar" />,
   },
 };
 

src/plugins/kibana_react/public/context/context.tsx

@@ -42,11 +42,11 @@ export const useKibana = <Extra extends object = {}>(): KibanaReactContextValue<
 export const withKibana = <Props extends { kibana: KibanaReactContextValue<any> }>(
   type: React.ComponentType<Props>
 ): React.FC<Omit<Props, 'kibana'>> => {
-  const enhancedType: React.FC<Omit<Props, 'kibana'>> = (props: Omit<Props, 'kibana'>) => {
+  const EnhancedType: React.FC<Omit<Props, 'kibana'>> = (props: Omit<Props, 'kibana'>) => {
     const kibana = useKibana();
     return React.createElement(type, { ...props, kibana } as Props);
   };
-  return enhancedType;
+  return EnhancedType;
 };
 
 export const UseKibana: React.FC<{
@@ -69,7 +69,7 @@ export const createKibanaReactContext = <Services extends KibanaServices>(
     const oldValue = useKibana();
     const { value: newValue } = useMemo(
       () => createKibanaReactContext({ ...services, ...oldValue.services, ...newServices }),
-      Object.keys(services)
+      [services, oldValue, newServices]
     );
     return createElement(context.Provider as React.ComponentType<any>, {
       value: newValue,

src/plugins/kibana_react/public/ui_settings/use_ui_setting.test.tsx

@@ -106,7 +106,7 @@ describe('useUiSetting', () => {
 });
 
 describe('useUiSetting$', () => {
-  const TestConsumer$: React.FC<{
+  const TestConsumerX: React.FC<{
     setting: string;
     newValue?: string;
   }> = ({ setting, newValue = '' }) => {
@@ -126,7 +126,7 @@ describe('useUiSetting$', () => {
 
     ReactDOM.render(
       <Provider>
-        <TestConsumer$ setting="foo" />
+        <TestConsumerX setting="foo" />
       </Provider>,
       container
     );
@@ -143,7 +143,7 @@ describe('useUiSetting$', () => {
 
     ReactDOM.render(
       <Provider>
-        <TestConsumer$ setting="non_existing" />
+        <TestConsumerX setting="non_existing" />
       </Provider>,
       container
     );
@@ -159,7 +159,7 @@ describe('useUiSetting$', () => {
 
     ReactDOM.render(
       <Provider>
-        <TestConsumer$ setting="theme:darkMode" />
+        <TestConsumerX setting="theme:darkMode" />
       </Provider>,
       container
     );
@@ -174,7 +174,7 @@ describe('useUiSetting$', () => {
 
     ReactDOM.render(
       <Provider>
-        <TestConsumer$ setting="a" newValue="c" />
+        <TestConsumerX setting="a" newValue="c" />
       </Provider>,
       container
     );

src/plugins/kibana_react/public/ui_settings/use_ui_setting.ts

@@ -65,6 +65,7 @@ export const useUiSetting$ = <T>(key: string, defaultValue?: T): [T, Setter<T>]
   const observable$ = useMemo(() => services.uiSettings!.get$(key, defaultValue), [
     key,
     defaultValue,
+    services.uiSettings,
   ]);
   const value = useObservable<T>(observable$, services.uiSettings!.get(key, defaultValue));
   const set = useCallback((newValue: T) => services.uiSettings!.set(key, newValue), [key]);

src/plugins/kibana_react/public/util/use_unmount.ts

@@ -20,5 +20,5 @@
 import { useEffect } from 'react';
 
 export function useUnmount(fn: () => void): void {
-  useEffect(() => fn, []);
+  useEffect(() => fn, [fn]);
 }

src/plugins/kibana_utils/public/store/react.ts

@@ -86,10 +86,12 @@ export const createContext = <
     comparator?: Comparator<Result>
   ): Result => {
     const { state$, get } = useStore();
+    /* eslint-disable react-hooks/exhaustive-deps */
     const [observable$, unsubscribe] = useMemo(
       () => observableSelector(get(), state$, selector, comparator),
       [state$]
     );
+    /* eslint-enable react-hooks/exhaustive-deps */
     useLayoutEffect(() => unsubscribe, [observable$, unsubscribe]);
     const value = useObservable(observable$, selector(get()));
     return value;