Anton Agestam
Anton Agestam
> Is that something really important? Yes, I consider it fundamental. > but I also have some requirements to make threads work in a short amount of time. I appreciate...
@schabowsky Hi, sorry for the slow response. As with this change I see no way of outputting this log message using the CLI I don't think it's mergable as is....
@lepilepi Hmm, yeah looks like django-storages has removed `.gzip` from the S3 class. So that would have to be fixed in a separate PR first. The expected behavior for failures...
> which was entirely swallowed by the ThreadPoolExecutor().map() so in the end there were no errors displayed, marked as success while missing several files. Yeah no I agree, I'm saying...
@Urth This is awesome, thanks! I'll try to look into what it takes to setup a Swift backend to test against, that will take some time.
@theilgaard Interesting. I'd be happy to review a PR that fixes this, provided it includes a regression test. Unfortunately I won't have time to work on this myself any time...
I'll review PRs implementing this.
Because `ManifestFilesMixin` was not around when collectfast was first built and support for it has never been implemented. Since integration with `ManifestFilesMixin` has never been supported there can't be bugs...
Thanks for putting in the work on this! I ran the test suite locally on your branch the the suite is passing. It would be really nice to maintain support...
This is not a known issue to me, do feel free to address this. collectfast has "live" tests uploading to real buckets, so perhaps those could be adjusted to also...