chore: remove legacy deps, deduplicate tests, update infra#220
chore: remove legacy deps, deduplicate tests, update infra#220olegshmuelov wants to merge 6 commits intofeat/compounding-supportfrom
Conversation
19d28d7 to
a009c4c
Compare
|
Can we please split this unit into multiple PRs? I think we have 5 distinct concerns here, review is really hard and I worry that we're entering some rollback risky waters here. Proposed split: 5 focused PRsPR A — fix: use system CA bundle by default
PR B — chore: replace bloxapp/ssv/logging with direct zap
PR C — chore: update CI/CD and linting infrastructure
PR D — chore: deduplicate integration test setup
PR E — chore: housekeeping and minor code cleanups
Recommended Merge Order
|
Thanks for the detailed breakdown. The challenge with splitting is that key changes span multiple of the proposed PRs. In terms of risk - there are no production logic changes here. The scope is:
I'd prefer not to prolong this as we have downstream work (0x02 withdrawal credentials) blocked on this branch. That said, happy to walk you through any section or address any specific concerns you raise. |
Right, I'll be focusing on logic part then |
Summary
go-tasks.ymlwith modern CI (ci.yml) and updatereleases.yml.golangci.ymlv2 config.gitignore,Dockerfile,Makefile, regenerate TLS test certsTest plan