search

diogo-fernan / malsub

A Python RESTful API framework for online malware analysis and threat intelligence services.
Other
368 stars 80 forks source link

Fix/remove comments #33

Closed alex-ilgayev closed 2 years ago

alex-ilgayev commented 2 years ago

Some of the heavy code comments make it hard to make internal changes. I took the step of removing completely all code comments, together with running black formatting and removing unused imports.

@diogo-fernan please go over it, and let me know if there are stuff you wish would stay. the black formatting is the first step for creating a nice pipeline for MR with testing and formatting check.

The functionality remained unchanged.

alex-ilgayev commented 2 years ago

Hey, @diogo-fernan thanks for the in-depth response! I commented on all your notes and fixed some of them. Please let me know if it is ok the way it is now.

diogo-fernan commented 2 years ago

Looks good to me! Thanks @alex-ilgayev.