Clarify (dotnet#4596)

patilb · May 31, 2021 · a1a9ae3 · a1a9ae3
1 parent be1295d
commit a1a9ae3
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 23 deletions.
diff --git a/.github/actions/status-checker/dist/index.js b/.github/actions/status-checker/dist/index.js
diff --git a/.github/actions/status-checker/dist/index.js.map b/.github/actions/status-checker/dist/index.js.map
diff --git a/.github/actions/status-checker/src/status-checker.ts b/.github/actions/status-checker/src/status-checker.ts
@@ -96,17 +96,7 @@ export async function checkStatus(token: string) {
 
     if (buildStatus.state == 'success') {
       if (buildStatus.description == 'Validation status: warnings') {
-        console.log('Found build warnings.');
-        core.setFailed('Please fix build warnings before merging.');
-
-        // return await octokit.repos.createCommitStatus({
-        //   owner: owner,
-        //   repo: repo,
-        //   sha: commit,
-        //   state: 'failure',
-        //   context: 'Check for build warnings',
-        //   description: 'Please fix build warnings before merging.',
-        // })
+        core.setFailed("Please fix OPS build warnings before merging. To see the warnings, click 'Details' next to the OpenPublishing.build status check at the bottom of your pull request.");
       }
       else {
         console.log("OpenPublishing.Build status check does not have warnings.");