freedomofpress / securethenews

An automated scanner and web dashboard for tracking TLS deployment across news organizations
https://securethe.news
GNU Affero General Public License v3.0
101 stars 29 forks source link

Remove safety-check related tools from pipenv #378

Closed chigby closed 2 years ago

chigby commented 2 years ago

This change removes flake8, safety and bandit from Pipenv and adjusts the makefile (and related scripts) to run them either inside a docker container or directly in the CI environment.

The reason for doing this is a recent update to Pipenv seemed to be breaking the make commands. Not fully sure what was going on with it but it seems like we want to move away from Pipenv anyway and it's not too hard to remove these packages from it.