fix: Disable a React Hooks ESLint check in App.tsx

georgefst · georgefst · commit cdbcb4a118a5 · 2022-07-14T15:49:56.000+01:00
diff --git a/packages/primer-app/src/App.tsx b/packages/primer-app/src/App.tsx
@@ -12,6 +12,9 @@ const App = (): JSX.Element => {
       <Error string={"No session id parameter: " + JSON.stringify(params)} />
     );
   }
+  // This hook is *technically* conditional.
+  // But if the condition above fails, then the app is broken anyway.
+  // eslint-disable-next-line react-hooks/rules-of-hooks
   const queryRes = useGetApiProgram({ session });
 
   if (!queryRes.isSuccess) {

Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,9 @@ const App = (): JSX.Element => {`
`12`	`12`	`<Error string={"No session id parameter: " + JSON.stringify(params)} />`
`13`	`13`	`);`
`14`	`14`	`}`
	`15`	`+ // This hook is technically conditional.`
	`16`	`+ // But if the condition above fails, then the app is broken anyway.`
	`17`	`+ // eslint-disable-next-line react-hooks/rules-of-hooks`
`15`	`18`	`const queryRes = useGetApiProgram({ session });`
`16`	`19`
`17`	`20`	`if (!queryRes.isSuccess) {`