Tom
Tom
Please, provide a PR with the missing type files and I will happily merge it into filetype.py.
You are reusing a readable stream that has been already read for the first 10 bytes. The `requests` package `response.content` implements an [iterable stream](https://github.com/psf/requests/blob/master/requests/models.py#L829) that flushes the internal buffer once...
Please, fix the CI build. There is a linter issue: https://travis-ci.org/h2non/filetype.py/jobs/377604223#L533-L539
I think you are right. In that case, I don't think that should bè implémentés within the context if this library.
Definitively, the public interface of those functions should be managed as part of the package's public API contract.
Happy to merge it :)
That might be true, but without metrics we don't know. Also, IMO the library is considerably fast enough for the 99% of the use cases. Why do you care of...
Interesting... my impression is that this is a CPython limitation, more than an implementation performance issue, but we can try improving things. If you can lead this by preparaing some...
I would suggest that a performance for this scenarios test should not involve any I/O at all. That would make the performance testing goal inaccurate, and therefore irrelevant. Instead, the...
Yeah, click looks neat, but for what we need here `argparse` is enough and built-in the stdlib.