mdolab / .github

0 stars 3 forks source link

Update azure_tapenade.yaml #26

Closed eirikurj closed 3 years ago

eirikurj commented 3 years ago

Purpose

Script should now fail if tapenade generated files are different.

Type of change

Testing

Checklist

ewu63 commented 3 years ago

I'm like 80% sure that this is not needed---if the last line of the script throws an error then Azure should fail. This is how we currently test flake8/black etc. without wrapping these. The issue with the IDwarp PR has to do with me being dumb -- I have reverted one commit that was causing files to not be generated.

eirikurj commented 3 years ago

Yes, it should not be needed, as it's the last line. I just wanted to make it explicit, as I thought something strange was happening. But let's not, as this adds what is effectively a redundant code. Closing