Update espresso-ci scripts

[jngrad]

Description of changes:

• use the new GET syntax for the GitHub API (fixes Update espresso-ci bot with new GET syntax #3612)
• print a clear error message and exit Python with a non-zero return value when the GitHub API returns an HTTP error code (partial fix for HTTP status codes #2832)

[KaiSzuttor]

LGTM modulo tmp changes

[jngrad]

I should have seen that coming: we need a chained if-else to test all possible names for clang-format and we need to use clang-format-9 instead of 6. The difference is minimal (only 2 files) but it's a circular change with version 6, i.e. once formatted with version 9, formatting it with the version 6 installed at the ICP will revert the change... So we're stuck with the Ubuntu 16.04 image until the ICP migrates to Ubuntu 20.04, which might take us months.

[codecov]

Codecov Report

Merging #3613 into python will decrease coverage by <1%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           python   #3613   +/-   ##
======================================
- Coverage      87%     87%   -1%     
======================================
  Files         525     525           
  Lines       23398   23398           
======================================
- Hits        20583   20582    -1     
- Misses       2815    2816    +1

Impacted Files	Coverage Δ
src/core/particle_data.cpp	96% <0%> (-1%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9ca0196...0289d77. Read the comment docs.

[KaiSzuttor]

I already approved

[fweik]

Yes, but there were new changes after, did you look at them?

[fweik]

Also, since when is the approval not removed if new changes are pushed?

[jngrad]

Also, since when is the approval not removed if new changes are pushed?

I realized that behavior several weeks ago and found it strange too, but did not question it. The "marked this pull request as ready for review 11 hours ago" message it posted automatically when removing the draft status of a PR. I did not add the automerge label immediately because we had to wait for the PR to complete to make sure we could trigger the webhook for a successful CI pipeline.

[fweik]

It's a setting in the branch protection rules. Not sure if this conflicts with kodiak's merge request commits, if so it has to be deactivated again.