move AWSS3 into extension

[lazarusA]

this PR moves AWSS3 (AWS) into an extension.

[lazarusA]

@meggart @mkitti, here, the extension. CI shows that is ok, however tests locally for MiniIO failed for me. Hopefully all is good.

[coveralls]

Pull Request Test Coverage Report for Build 19576861595

Details

• 12 of 12 (100.0%) changed or added relevant lines in 3 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.03%) to 89.387%

---

Totals
Change from base Build 19542891216:	0.03%
Covered Lines:	977
Relevant Lines:	1093

---

💛 - Coveralls

[mkitti]

The MinIO tests are a bit flimsy in my experience as well. Didbm they work better for you previously?

[lazarusA]

I also updated a test issue that I also fixed in #223

Fletcher32Filter: Error During Test at /home/runner/work/Zarr.jl/Zarr.jl/test/Filters.jl:7
  Got exception outside of a @test
  InexactError: trunc(UInt8, 256)
  Stacktrace:

latest commit fixes this.

[lazarusA]

The MinIO tests are a bit flimsy in my experience as well. Didbm they work better for you previously?

I think so, not sure anymore 😄 .

[mkitti]

This looks good to merge to me.

[mkitti]

Do you want to merge this first and then #223 or the other way around?

[lazarusA]

Do you want to merge this first and then #223 or the other way around?

please merge this one first. The merge button still doesn show for me. But its ready !

[nhz2]

Is this a breaking change?
Also curious why an extension vs a subdir package?

This looks good to me as well.

[lazarusA]

Is this a breaking change? Also curious why an extension vs a subdir package?

This looks good to me as well.

yes, all these, also #223, will be a breaking change. I will add CHANGELOG.md file in a separated PR before tagging the next release.