more eslint fixes

samples/react-add-js-css-ref/.eslintrc.js

@@ -88,7 +88,7 @@ module.exports = {
         //                    or else return the object to a caller (who assumes this responsibility).  Unterminated
         //                    promise chains are a serious issue.  Besides causing errors to be silently ignored,
         //                    they can also cause a NodeJS process to terminate unexpectedly.
-        "@typescript-eslint/no-floating-promises": 0,
+        "@typescript-eslint/no-floating-promises": 2,
         // RATIONALE:         Catches a common coding mistake.
         "@typescript-eslint/no-for-in-array": 2,
         // STANDARDIZED BY:   @typescript-eslint\eslint-plugin\dist\configs\recommended.json
@@ -295,7 +295,7 @@ module.exports = {
         // STANDARDIZED BY:   @typescript-eslint\eslint-plugin\dist\configs\recommended.json
         "no-var": 2,
         // RATIONALE:         Generally not needed in modern code.
-        "no-void": 1,
+        "no-void": 0,
         // RATIONALE:         Obsolete language feature that is deprecated.
         // STANDARDIZED BY:   eslint\conf\eslint-recommended.js
         "no-with": 2,

samples/react-add-js-css-ref/src/webparts/addJsCssReference/AddJsCssReferenceWebPart.ts

@@ -3,7 +3,6 @@ import * as ReactDom from 'react-dom';
 import { Version } from '@microsoft/sp-core-library';
 import { BaseClientSideWebPart } from "@microsoft/sp-webpart-base";
 import { IPropertyPaneConfiguration } from "@microsoft/sp-property-pane";
-import * as strings from 'AddJsCssReferenceWebPartStrings';
 import AddJsCssReference, { IAddJsCssReferenceProps } from './components/AddJsCssReference';
 import { getSP } from './pnpjsConfig';
 

samples/react-add-js-css-ref/src/webparts/addJsCssReference/components/AddJsCssReference.tsx

@@ -290,20 +290,20 @@ export default class AddJsCssReference extends React.Component<IAddJsCssReferenc
     );
   }
 
-  public  componentDidMount(): void {
-     this.checkPermisson();
+  public componentDidMount() {
+    void this.checkPermisson();
   }
 
-
   private async checkPermisson() {
     const perms2 = await this._sp.web.currentUserHasPermissions(PermissionKind.ManageWeb);
-    this.setState({ userHasPermission: perms2 });
     console.log(perms2);
-    this.setState({ showspinner: false });
+    this.setState({
+      userHasPermission: perms2,
+      showspinner: false
+    });
     if (perms2) {
-      await this.getCustomAction();
+      void this.getCustomAction();
     }
-
   }