eslint rule change

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

@@ -87,7 +87,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": 2,
+        "@typescript-eslint/no-floating-promises": 0,
         // RATIONALE:         Catches a common coding mistake.
         "@typescript-eslint/no-for-in-array": 2,
         // STANDARDIZED BY:   @typescript-eslint\eslint-plugin\dist\configs\recommended.json

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

@@ -25,11 +25,13 @@ export default class AddJsCssReferenceWebPart extends BaseClientSideWebPart<IAdd
 
     ReactDom.render(element, this.domElement);
   }
-protected async onInit(): Promise<void> {
-  await super.onInit();
-  getSP(this.context);
 
+  protected async onInit(): Promise<void> {
+    // eslint-disable-next-line @typescript-eslint/no-floating-promises
+    super.onInit();
+    getSP(this.context);
   }
+
   protected onDispose(): void {
     ReactDom.unmountComponentAtNode(this.domElement);
   }

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

@@ -2,15 +2,11 @@ import { WebPartContext } from "@microsoft/sp-webpart-base";
 
 // import pnp and pnp logging system
 import { spfi, SPFI, SPFx } from "@pnp/sp";
-import "@pnp/sp/webs";
-import "@pnp/sp/lists";
-import "@pnp/sp/items";
-import "@pnp/sp/batching";
 
 let _sp: SPFI = null;
 
 export const getSP = (context?: WebPartContext): SPFI => {
-  if (context !== null) {
+  if (context) {
     _sp = spfi().using(SPFx(context));
   }
   return _sp;