MNT minor linter bot message enhancements (scikit-learn#26670)

Author: adrinjalali

.github/workflows/lint.yml

@@ -36,11 +36,14 @@ jobs:
           pip install pytest ruff $(get_dep mypy min) $(get_dep black min) cython-lint
 
       - name: Run linting
+        id: lint-script
         # We download the linting script from main, since this workflow is run
         # from main itself.
         run: |
-          curl https://raw.githubusercontent.com/scikit-learn/scikit-learn/main/build_tools/linting.sh -o ./build_tools/linting.sh
+          set +e
+          curl https://raw.githubusercontent.com/scikit-learn/scikit-learn/main/build_tools/linting.sh --retry 5 -o ./build_tools/linting.sh
           ./build_tools/linting.sh &> /tmp/linting_output.txt
+          cat /tmp/linting_output.txt
 
       - name: Upload Artifact
         if: always()

build_tools/get_comment.py

@@ -152,30 +152,44 @@ def get_message(log_file, repo, pr_number, sha, run_id, details):
         details=details,
     )
 
-    commit_link = (
-        "\n\n_Generated for commit:"
-        f" [{sha[:7]}](https://github.com/{repo}/pull/{pr_number}/commits/{sha})_"
+    sub_text = (
+        "\n\n<sub> _Generated for commit:"
+        f" [{sha[:7]}](https://github.com/{repo}/pull/{pr_number}/commits/{sha}). "
+        "Link to the linter CI: [here]"
+        f"(https://github.com/{repo}/actions/runs/{run_id})_ </sub>"
     )
 
     if not message:
         # no issues detected, so this script "fails"
         return (
             "## ✔️ Linting Passed\n"
             "All linting checks passed. Your pull request is in excellent shape! ☀️"
-            + commit_link
+            + sub_text
         )
 
+    if not details:
+        # This happens if posting the log fails, which happens if the log is too
+        # long. Typically, this happens if the PR branch hasn't been updated
+        # since we've introduced import sorting.
+        branch_not_updated = (
+            "_Merging with `upstream/main` might fix / improve the issues if you "
+            "haven't done that since 21.06.2023._\n\n"
+        )
+    else:
+        branch_not_updated = ""
+
     message = (
         "## ❌ Linting issues\n\n"
-        "This PR is introducing linting issues. Here's a summary of the issues. "
-        "Note that you can avoid having linting issues by enabling `pre-commit` "
-        "hooks. Instructions to enable them can be found [here]("
-        "https://scikit-learn.org/dev/developers/contributing.html#how-to-contribute)."
-        "\n\n"
-        "You can see the details of the linting issues under the `lint` job [here]"
-        f"(https://github.com/{repo}/actions/runs/{run_id})\n\n"
+        + branch_not_updated
+        + "This PR is introducing linting issues. Here's a summary of the issues. "
+        + "Note that you can avoid having linting issues by enabling `pre-commit` "
+        + "hooks. Instructions to enable them can be found [here]("
+        + "https://scikit-learn.org/dev/developers/contributing.html#how-to-contribute)"
+        + ".\n\n"
+        + "You can see the details of the linting issues under the `lint` job [here]"
+        + f"(https://github.com/{repo}/actions/runs/{run_id})\n\n"
         + message
-        + commit_link
+        + sub_text
     )
 
     return message