update comments

Author: abdulahmad307

.github/actions/find/src/findForUrl.ts

@@ -39,6 +39,8 @@ export async function findForUrl(
     //   in unit tests on whether 'loadPlugins' was called or not
     // - alternatively, we can wrap the 'plugin.default(...)' call in another function
     //   and make assertions on whether that function was called or not
+    // - the other option is to wrap each plugin in a class instance
+    //   and make assertions on something like 'plugin.run' being called or not
     if (scansContext.shouldRunPlugins) {
       const plugins = await loadPlugins()
       for (const plugin of plugins) {
@@ -56,7 +58,6 @@ export async function findForUrl(
       screenshotId = await generateScreenshots(page)
     }
 
-    console.log('rawFindings: ', rawFindings)
     findings =
       rawFindings?.violations.map(violation => ({
         scannerType: 'axe',

.github/actions/find/src/scansContextProvider.ts

@@ -10,11 +10,11 @@ let scansContext: ScansContext | undefined
 export function getScansContext() {
   if (!scansContext) {
     const scansJson = core.getInput('scans', {required: false})
-    console.log('scans input: ', scansJson)
     const scansToPerform = JSON.parse(scansJson || '{}')
     // - if we dont have a scans input
     //   or we do have a scans input, but it only has 1 item and its 'axe'
     //   then we only want to run 'axe' and not the plugins
+    // - keep in mind, 'onlyAxeScan' is not the same as 'shouldPerformAxeScan'
     const onlyAxeScan = scansToPerform.length === 0 || (scansToPerform.length === 1 && scansToPerform[0] === 'axe')
 
     scansContext = {