[riot-notifications] [RIOT-OS/RIOT] dist/tools/codespell: annotate errors in Github Action (#15751)

José Alamos notifications at github.com
Wed Jan 13 15:09:37 CET 2021

@jia200x commented on this pull request.

+            ANNOTATION="There is a typo: ${ERROR}\n"
+            ANNOTATION="${ANNOTATION}\nIf this is a false positive,"
+            ANNOTATION="${ANNOTATION} add it to ${IGNORED_WORDS}."
+            ANNOTATION="${ANNOTATION}\n\nYou can fix this interactively by"
+            ANNOTATION="${ANNOTATION} calling\n\n"
+            github_annotate_error "${FILENAME}" "${LINENUM}" "${ANNOTATION}"
+        done
+    else
+        printf "%sThere are typos in the following files:%s\n\n" "${CERROR}" "${CRESET}"
+        printf "%s\n" "${ERRORS}"
+        printf "If those are false positives, add them to %s\n" "${RIOTTOOLS}/codespell/ignored_words.txt"

I'm not sure if we need to show the absolute RIOTTOOLS patch here. In the annotations it looks like "/data/riotbuild/dist/tools/codespell/ignored_words.txt.".

I think it would be enough to point relative to RIOTBASE

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210113/73edee65/attachment.htm>

More information about the notifications mailing list