Contributing and testing housekeeping

elixir-mint / mint

Functional HTTP client for Elixir with support for HTTP/1 and HTTP/2 🌱

Apache License 2.0

1.36k stars 106 forks source link

Contributing and testing housekeeping #440

Closed kianmeng closed 4 days ago

kianmeng commented 4 days ago

List of changes:

autolink Apache license and switch to https
bump Alpine Linux image from 3.17.2 to 3.20.1
exclude :integration test by default
fix cannot build Docker images with Podman Compose
fix incorrect CI badge which still point to Travis CI
fix markups in readme by prettier
replace :requires_internet_connection with :integration tag for consistentcy
tag integration test module in http1 with :integration
update console commands markdown so we can click and copy them

coveralls commented 4 days ago

Pull Request Test Coverage Report for Build d65f6218de2b0338480e3dbad688d436a946b5ed-PR-440

Details

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.389%

Totals
Change from base Build d3070089d4d22e8062e94df1334889437658fdee:	0.0%
Covered Lines:	1275
Relevant Lines:	1459

💛 - Coveralls

kianmeng commented 4 days ago

This is a lot of different changes 🙃 In general, I'd recommend to split these out in multiple PRs next time so that we can pick and choose. I left a few blocking comments about stuff that I would prefer not to change. Thanks for the contribution!

I will close this PR and submit smaller PRs instead with the changes you intend to keep. It's easier that way.