[llvm] [CI] Make premerge_advisor_explain write comments (PR #166605)

[David Spickett via llvm-commits]

================
@@ -33,17 +33,18 @@ function at-exit {
   # If building fails there will be no results files.
   shopt -s nullglob
 
-  if [[ "$GITHUB_STEP_SUMMARY" != "" ]]; then
+  if [[ "$GITHUB_ACTIONS" != "" ]]; then
     python "${MONOREPO_ROOT}"/.ci/generate_test_report_github.py \
       $retcode "${BUILD_DIR}"/test-results.*.xml "${MONOREPO_ROOT}"/ninja*.log \
       >> $GITHUB_STEP_SUMMARY
+    python "${MONOREPO_ROOT}"/.ci/premerge_advisor_explain.py \
+      $(git rev-parse HEAD~1) $retcode ${{ secrets.GITHUB_TOKEN }} \
----------------
DavidSpickett wrote:

Currently CI is failing on this line, is that expected?

Ah, it is I think. As right now CI is running this as code from a PR and therefore won't give it such tokens or permissions. Right?

Once it's merged it'll be trusted and be given the token.

https://github.com/llvm/llvm-project/pull/166605