Merge pull request #1125 from rust-lang/eslint-and-more

Upgrade JS dependencies: semver-compatible, ESlint, uuid

Author: shepmaster

ui/frontend/.eslintrc.js

@@ -12,7 +12,6 @@ node_modules
 *.scss
 
 # Slowly migrate files that we've touched
-!.eslintrc.js
 !BuildMenu.tsx
 !ButtonSet.tsx
 !Header.tsx
@@ -32,6 +31,7 @@ node_modules
 !editor/MonacoEditorCore.tsx
 !editor/SimpleEditor.tsx
 !editor/rust_monaco_def.ts
+!eslint.config.mjs
 !hooks.ts
 !observer.ts
 !prism-shim.ts

ui/frontend/.prettierignore

@@ -11,7 +11,7 @@ interface EitherProps<T extends string> extends ConfigElementProps {
   aLabel?: string;
   bLabel?: string;
   value: T;
-  onChange: (_: T) => any;
+  onChange: (_: T) => void;
 }
 
 export const Either =
@@ -39,7 +39,7 @@ export const Either =
 interface SelectProps<T extends string> extends ConfigElementProps {
   children: React.ReactNode;
   value: T;
-  onChange: (_: T) => any;
+  onChange: (_: T) => void;
 }
 
 export const Select = <T extends string,>({ value, onChange, children, ...rest }: SelectProps<T>) => (

ui/frontend/ConfigElement.tsx

@@ -2,8 +2,6 @@ import React from 'react';
 
 import * as styles from './Icon.module.css';
 
-/* eslint-disable max-len */
-
 // These icons came from Material Design originally
 // https://material.io/tools/icons/?icon=assignment&style=outline
 
@@ -30,7 +28,6 @@ export const MoreOptionsIcon = () => (
 
 export const MoreOptionsActiveIcon = () => (
   <svg className={styles.icon} height="18" viewBox="0 0 24 24" width="18" xmlns="http://www.w3.org/2000/svg">
-    {/* eslint-disable-next-line react/no-unknown-property */}
     <path fillRule="evenodd" fill="#428bca" d="M4,5 h16 a3,3 0 0,1 3,3 v8 a3,3 0 0,1 -3,3 h-16 a3,3 0 0,1 -3,-3 v-8 a3,3 0 0,1 3,-3 Z M6 10c-1.1 0-2 .9-2 2s.9 2 2 2 2-.9 2-2-.9-2-2-2zm12 0c-1.1 0-2 .9-2 2s.9 2 2 2 2-.9 2-2-.9-2-2-2zm-6 0c-1.1 0-2 .9-2 2s.9 2 2 2 2-.9 2-2-.9-2-2-2z" />
   </svg>
 );

ui/frontend/Icon.tsx

@@ -31,7 +31,7 @@ interface TabProps {
   kind: Focus;
   focus?: Focus;
   label: string;
-  onClick: () => any;
+  onClick: () => void;
   tabProps: object;
 }
 

ui/frontend/Output.tsx

@@ -24,7 +24,7 @@ const Execute: React.FC = () => {
 };
 
 interface WarningProps {
-  addMainFunction: () => any;
+  addMainFunction: () => void;
 }
 
 const Warning: React.FC<WarningProps> = props => (

ui/frontend/Output/Execute.tsx

@@ -1,8 +1,8 @@
 import React from 'react';
-import { UnknownAction } from '@reduxjs/toolkit';
+import { Reducer, UnknownAction } from '@reduxjs/toolkit';
 
 import { createBrowserHistory as createHistory, Path, Location } from 'history';
-import { createRouter, PlainOrThunk } from './uss-router';
+import { createRouter, PlainOrThunk, RouterObject, StoreArg } from './uss-router';
 import UssRouter from './uss-router/Router';
 
 import qs from 'qs';
@@ -82,7 +82,7 @@ const locationToAction = (location: Location): PlainOrThunk<State, UnknownAction
 };
 
 export default class Router extends React.Component<RouterProps> {
-  private router: any;
+  private router: RouterObject<UnknownAction>;
 
   public constructor(props: RouterProps) {
     super(props);
@@ -104,6 +104,6 @@ export default class Router extends React.Component<RouterProps> {
 
 interface RouterProps {
   children: React.ReactNode;
-  store: any;
-  reducer: any;
+  store: StoreArg<State, UnknownAction>;
+  reducer: Reducer<State>;
 }

ui/frontend/Router.tsx

@@ -7,7 +7,7 @@ interface SelectOneProps<T> {
   name: string;
   currentValue: T;
   thisValue: T;
-  changeValue: (_: T) => any;
+  changeValue: (_: T) => void;
 }
 
 export default class SelectOne<T> extends React.PureComponent<SelectOneProps<T>> {

ui/frontend/SelectOne.tsx

@@ -23,7 +23,9 @@ export function jsonGet(url: FetchArg): Promise<unknown> {
   });
 }
 
-export function jsonPost(url: FetchArg, body: Record<string, any>): Promise<unknown> {
+type ToJson = Parameters<typeof JSON.stringify>[0];
+
+export function jsonPost(url: FetchArg, body: ToJson): Promise<unknown> {
   return fetchJson(url, {
     method: 'post',
     body: JSON.stringify(body),

ui/frontend/api.ts

@@ -32,7 +32,7 @@ interface Props {
 }
 
 interface CompileActions {
-  action: AsyncThunk<CompileResponseBody, CompileRequestBody, {}>;
+  action: AsyncThunk<CompileResponseBody, CompileRequestBody, object>;
   performCompile: () => ThunkAction;
 }