secure-software-engineering / phasar

A LLVM-based static analysis framework.
Other
930 stars 140 forks source link

Feature/statistics update #524

Closed janniclas closed 1 year ago

janniclas commented 1 year ago

This merge request contains

janniclas commented 1 year ago

Extended the request with a new GitHub action to build the docker image and publish it to GitHubs container registry. This can greatly ease the initial adoption of PhASAR imho. It would be even better if we also push the image to the official docker registry, if you think this is desirable just reach out to me and I can walk you through the necessary steps @pdschubert .

pdschubert commented 1 year ago

Hey @vulder can you tell why the pre-commit fails here? And if so, how to fix it?

Thanks! Philipp

vulder commented 1 year ago

@pdschubert The hook most often tells you the issue: trim trailing whitespace.................................................Failed After the docker line are trailing whitespaces, remove them and the check will be happy again :grinning: