From d1e66f0446dab127a10e51dc073edd030182d9d7 Mon Sep 17 00:00:00 2001
From: Jian Chen <cjian@microsoft.com>
Date: Wed, 15 May 2024 16:41:59 -0400
Subject: [PATCH]  Increase NPM ComponentDetection.Timeout: 1200 (#20681)

### Description
<!-- Describe your changes. -->



### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
---
 .../github/azure-pipelines/npm-packaging-pipeline.yml         | 4 ++--
 .../component-governance-component-detection-steps.yml        | 3 ++-
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/tools/ci_build/github/azure-pipelines/npm-packaging-pipeline.yml b/tools/ci_build/github/azure-pipelines/npm-packaging-pipeline.yml
index 99d6b416d2..9d5628fd36 100644
--- a/tools/ci_build/github/azure-pipelines/npm-packaging-pipeline.yml
+++ b/tools/ci_build/github/azure-pipelines/npm-packaging-pipeline.yml
@@ -10,10 +10,10 @@ parameters:
   default: 'nightly (@dev)'
 
 variables:
-  # pipeline should define the following varaibles
+  # pipeline should define the following variables
   #   ExtraBuildArgs
   #   VersionSuffix
-
+  ComponentDetection.Timeout: 1800
   ${{ if eq(parameters.NpmPublish, 'nightly (@dev)') }}:
     NpmPackagingMode: 'dev'
   ${{ if eq(parameters.NpmPublish, 'release candidate (@rc)') }}:
diff --git a/tools/ci_build/github/azure-pipelines/templates/component-governance-component-detection-steps.yml b/tools/ci_build/github/azure-pipelines/templates/component-governance-component-detection-steps.yml
index 279cf40d47..aa9f6826af 100644
--- a/tools/ci_build/github/azure-pipelines/templates/component-governance-component-detection-steps.yml
+++ b/tools/ci_build/github/azure-pipelines/templates/component-governance-component-detection-steps.yml
@@ -12,6 +12,7 @@ steps:
 
   - task: ms.vss-governance-buildtask.governance-build-task-component-detection.ComponentGovernanceComponentDetection@0
     displayName: 'Component Detection'
+    continueOnError: true
     condition:
       or(or(and(eq('${{parameters.condition}}', 'ci_only'), and(succeeded(), in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI', 'Scheduled'))),
           and(eq('${{parameters.condition}}', 'always'), always())),
@@ -20,7 +21,7 @@ steps:
       # ignore unit tests in emscripten. emscripten unit tests are not used in onnxruntime build
       # ignore onnx third_party directory. onnx third_party directory is not responsible for onnxruntime build
       # ignore react_native e2e node_modules directory. react_native e2e node_modules directory is generated by react_native e2e tests
-        # ignore github directory. github directory is used for github actions, not for onnxruntime package
+      # ignore github directory. github directory is used for github actions, not for onnxruntime package
       ignoreDirectories:
         '$(Build.SourcesDirectory)/cmake/external/emsdk/upstream/emscripten/tests,
         $(Build.SourcesDirectory)/cmake/external/onnx/third_party,