search

chaoss / grimoirelab-graal

A Generic Repository AnALyzer
GNU General Public License v3.0
21 stars 62 forks source link

[graal] Remove unused config_file option #11

Closed aswanipranjal closed 5 years ago

aswanipranjal commented 5 years ago

I ran graal from the terminal and the following warning popped up:

Screenshot 2019-03-10 at 6 14 39 PM

This code is about fixing that warning by using ConfigParser class instead of SafeConfigParser.

EDIT: Removed the unused code to pass a config file through command line while running graal.

valeriocos commented 5 years ago

The function read_config_file can be removed and the corresponding code in the main and parse_args. This code comes from Perceval, but it wasn't used and was removed some time ago (https://github.com/chaoss/grimoirelab-perceval/commit/d655a8ec3f5eae852611de0a175b044461247f40#diff-c9fbcf2779da469d6957976cd938491a).

@aswanipranjal , could you modify your PR according to the changes above?

Thank you