search

digitalr00ts / fizzbuzzx

An unnecessarily over-engineered fizzbuzz solution
Apache License 2.0
2 stars 2 forks source link

Fix failing tests #47

Closed drts01 closed 3 years ago

drts01 commented 3 years ago

Pull request #46 introduced failing tests.

pelmini commented 3 years ago

Ill try to take a stab at this

pelmini commented 3 years ago

@xsc27 So, I took a look at the error that we were getting with the linting tests, and it seems to be an issue with the way we are calling the lint command? We get a ERROR: InvocationError for command /bin/sh -c 'flakehell lint *.py' (exited with code 1). Not sure how to fix this though. From my understanding of the command, are we basically just piping the flakehell lint command to a shell command interpreter, so that way we can run the flakehell lint command to all the python files we have?