par-tec / super-sast

A docker container with different testing tools.
BSD 3-Clause "New" or "Revised" License
12 stars 3 forks source link

Fixes #27 #31

Closed nocturn9x closed 1 year ago

nocturn9x commented 1 year ago

Context information is now added when copying Java validators fails (See #27). A generic error handler has also been added

nocturn9x commented 1 year ago

@ioggstream Let me know if you want the username to be printed as well, as that requires a bit of /etc/passwd hackery which I'm not sure is suitable for production.

ioggstream commented 1 year ago

Echo $USER , pwd and home dir should work for now.

ioggstream commented 1 year ago

Tests should work. Add me as a reviewer when tests pass

nocturn9x commented 1 year ago

The latest modifications I pushed should've fixed the conflicts with the CI pipeline. It took me longer than I expected due to some issues with poetry and pre-commit. Let me know if there's anything else that needs to be done here

ioggstream commented 1 year ago

@nocturn9x Merging, thanks! See comments for more details.