Skip to content

Commit

Permalink
Report error logs from codecov upload fail
Browse files Browse the repository at this point in the history
This patch extends the REPORT_ERROR_LOGS flag behavior to also report
the error log of failed codecov uploads.

I expect there's little we can do on our end, but more information will
at least help to understand the problem.
  • Loading branch information
marshallward committed Aug 12, 2022
1 parent 1f1b8ad commit 5b90789
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .testing/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -602,6 +602,9 @@ report.cov: run.cov codecov
2> build/cov/codecov.err \
&& echo -e "${MAGENTA}Report uploaded to codecov.${RESET}" \
|| { \
if [ "$(REPORT_ERROR_LOGS)" = true ]; then \
cat build/cov/codecov.err ; \
fi ; \
echo -e "${RED}Failed to upload report.${RESET}" ; \
if [ "$(REQUIRE_COVERAGE_UPLOAD)" = true ] ; then false ; fi ; \
}
Expand Down Expand Up @@ -740,6 +743,9 @@ report.cov.unit: build/unit/MOM_file_parser_tests.F90.gcov codecov
2> build/unit/codecov.err \
&& echo -e "${MAGENTA}Report uploaded to codecov.${RESET}" \
|| { \
if [ "$(REPORT_ERROR_LOGS)" = true ]; then \
cat build/unit/codecov.err ; \
fi ; \
echo -e "${RED}Failed to upload report.${RESET}" ; \
if [ "$(REQUIRE_COVERAGE_UPLOAD)" = true ] ; then false ; fi ; \
}
Expand Down

0 comments on commit 5b90789

Please sign in to comment.