Nathan Youngman
Nathan Youngman
Elixir 1.16.0 deprecated `File.stream!(file, options, line_or_bytes)` So this line `File.stream!(path, [], 2048)` should just be `File.stream!(path, 2048)`. I've had to make the same change in my own code to resolve...
FYI, I'm seeing a number of test failures (8) locally -- on the `master` branch as well.
@yordis Are there any other changes needed before we can merge this and publish a Tesla release that supports Elixir 1.16?
CI updated. I don't think we **need** any matrix exceptions based on the versions I saw over here, but I'm happy to make other changes. https://github.com/ex-aws/ex_aws_s3/pull/239/files Thanks.
Merged in master.
@yordis With Dialyzer passing on all Elixir versions, I hope we can review and merge this even without the httparrot issues fixed, as that would allow us to stop using...
Please let me know if you'd like any other changes. Thanks!
> Hopefully we can figure out the elixir 1.6+ situation next 😠Absolutely. I already opened a pull request to httparrot, so we'll see how that goes. @yordis We'd appreciate...
Looks like adding AppSignal to the library works, but this feels like there could be better documentation in one place or another. The Tesla readme doesn't even link to the...
I'm an idiot -- needed to add `{:telemetry, "~> 1.2"},` to the mix file (which AppSignal brought in automatically in the place I had it). But that should be documented,...