-
Notifications
You must be signed in to change notification settings - Fork 171
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix config #72
Fix config #72
Conversation
go go @alexcpsec |
So... many... PRs... |
Do you want me to stop working on Combine? |
LOL! No, keep up the good work ;) |
Why do we have SafeConfigParser on reaper.py and ConfigParser on the other ones? Can't they all be "safe"? |
ah, good catch, I was trying to figure something out. |
…he process. This is begging for refactoring (#68)
Made a change so every file checks for the .cfg file. As it were, Thresher and Baler would run anyway. |
This seems good to go on my end. Waiting your approval on my change. |
Good catch on baler (and the output). I'd left it out of Thresher because it doesn't use any configs and is therefore spurious. |
The issue is that then Thresher runs anyway and either runs in an old |
Eh, might as well keep the check since it's going to have configs eventually from #23 |
? |
Ready for testing
Implement changes in #68