DLA-Future icon indicating copy to clipboard operation
DLA-Future copied to clipboard

Add include with the help of inshpect

Open gulivarese opened this issue 1 year ago • 2 comments

gulivarese avatar Apr 30 '24 12:04 gulivarese

cscs-ci run

gulivarese avatar Apr 30 '24 12:04 gulivarese

@msimberg I've just finished with the review about the include file

gulivarese avatar May 08 '24 17:05 gulivarese

cscs-ci run

gulivarese avatar May 15 '24 14:05 gulivarese

Reminder, we should make the inshpect check required before merging this.

msimberg avatar May 21 '24 13:05 msimberg

Is it normal that the check takes so long? Currently 1h30 and still running.

Certainly not... It should take a couple of minutes at most. I can't seem to cancel the runs either (and check_format also seems to be stuck...). We may have to wait for the jobs to time out (hopefully) and retry. Looks like it may perhaps be an issue with the runners.

msimberg avatar May 21 '24 15:05 msimberg

Is it normal that the check takes so long? Currently 1h30 and still running.

Certainly not... It should take a couple of minutes at most. I can't seem to cancel the runs either (and check_format also seems to be stuck...). We may have to wait for the jobs to time out (hopefully) and retry. Looks like it may perhaps be an issue with the runners.

Apparently something is wrong with actions... even #1146 is having problems.

rasolca avatar May 21 '24 15:05 rasolca

cscs-ci run

rasolca avatar May 23 '24 14:05 rasolca