aptly-dev / aptly

aptly - Debian repository management tool
https://www.aptly.info/
MIT License
2.56k stars 374 forks source link

Switch to aws-sdk-go-v2 #1226

Closed lebauce closed 11 months ago

lebauce commented 11 months ago

Requirements

All new code should be covered with tests, documentation should be updated. CI should pass.

Description of the Change

This PR switches aptly from using aws-sdk-go to aws-sdk-go-v2, the new Go SDK for AWS. Among many improvement, one is that the endpoints are not stored in global variables, freeing some memory.

Checklist

codecov[bot] commented 11 months ago

Codecov Report

Merging #1226 (fc0b4b9) into master (322e5c1) will decrease coverage by 0.03%. The diff coverage is 59.03%.

@@            Coverage Diff             @@
##           master    #1226      +/-   ##
==========================================
- Coverage   65.96%   65.94%   -0.03%     
==========================================
  Files         143      143              
  Lines       16188    16178      -10     
==========================================
- Hits        10679    10668      -11     
  Misses       4756     4756              
- Partials      753      754       +1     
Files Coverage Δ
s3/public.go 55.95% <59.03%> (-1.57%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more