search

ARMmbed / mbed-os-tf-m-regression-tests

An Mbed OS application that runs the TF-M regression tests
2 stars 12 forks source link

psa_builder.py: catch command not found error #55

Closed LDong-Arm closed 3 years ago

LDong-Arm commented 4 years ago

@ARMmbed/mbed-os-security

Without catching this exception, are_dependencies_installed() won't have a chance to print a message before the script crashes.

LDong-Arm commented 4 years ago

Code format check failed in CI, but it doesn't say what format it expects, or what the formatter command is?

LDong-Arm commented 4 years ago

@evedon FYI

jainvikas8 commented 4 years ago

@LDong-Arm 

Run lgeiger/black-action@v1.0.1
  with:
    args: . -l 79 --check
LDong-Arm commented 4 years ago

@LDong-Arm

Run lgeiger/black-action@v1.0.1
  with:
    args: . -l 79 --check

It's a GitHub Action, triggered in a Docker container. Would be better to dump the format diff in CI, so users wouldn't need to install all the tools to reproduce it locally...

LDong-Arm commented 3 years ago

This is from over a month ago, shall we get it in?

LDong-Arm commented 3 years ago

Please rebase this PR... So that it can run the Travis job

Done. Let's wait for CI.