Attention is currently required from: Hoernchen, laforge.

Patch set 3:Code-Review +1

View Change

2 comments:

• Patchset:

  • Patch Set #2:

    I'd say this needs some more explanation and possibly integration? What is the purpose of the file? […]

    • pre-commit hook is there now
    • server side: needs clang-format added to the docker container first: https://gerrit.osmocom.org/c/docker-playground/+/39763, I can adjust contrib/jenkins.sh in a follow-up patch after both are merged so we don't need to block here

• File contrib/pre-commit-hook:

  • Patch Set #3, Line 1: #!/usr/bin/env bash

    maybe make this executable?

To view, visit change 39470. To unsubscribe, or for help writing mail filters, visit settings.