Skip to content

Comments

refactor(retry): extract delay calculation into strategy implementations#72

Merged
ns-vasilev merged 1 commit intomainfrom
refactor/strategies
Feb 22, 2026
Merged

refactor(retry): extract delay calculation into strategy implementations#72
ns-vasilev merged 1 commit intomainfrom
refactor/strategies

Conversation

@ns-vasilev
Copy link
Member

No description provided.

@ns-vasilev ns-vasilev self-assigned this Feb 22, 2026
@ns-vasilev ns-vasilev requested a review from a team as a code owner February 22, 2026 10:33
@spacecode-github-bot
Copy link

2 Warnings
⚠️

⚠️ Some source files are missing corresponding test files:

  • RetryIterator → expected test file: RetryIteratorTests.swift
  • ConstantDelayStrategy → expected test file: ConstantDelayStrategyTests.swift
  • ExponentialDelayStrategy → expected test file: ExponentialDelayStrategyTests.swift
⚠️ Public API changed but no documentation updates found

Generated by 🚫 Danger

@codecov
Copy link

codecov bot commented Feb 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.34%. Comparing base (784f9b0) to head (dea9208).
⚠️ Report is 3 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
+ Coverage   97.85%   99.34%   +1.50%     
==========================================
  Files          10       13       +3     
  Lines         650      610      -40     
==========================================
- Hits          636      606      -30     
+ Misses         14        4      -10     
Files with missing lines Coverage Δ
...Typhoon/Classes/Extensions/Double+SafeUInt64.swift 100.00% <100.00%> (ø)
...Classes/RetrySequence/Iterator/RetryIterator.swift 100.00% <100.00%> (+10.99%) ⬆️
.../Typhoon/Classes/RetrySequence/RetrySequence.swift 100.00% <100.00%> (ø)
...trySequence/Strategies/ConstantDelayStrategy.swift 100.00% <100.00%> (ø)
...Sequence/Strategies/ExponentialDelayStrategy.swift 100.00% <100.00%> (ø)
...Typhoon/Classes/Strategy/RetryPolicyStrategy.swift 100.00% <100.00%> (ø)
...phoonTests/UnitTests/RetryPolicyServiceTests.swift 100.00% <100.00%> (ø)
...hoonTests/UnitTests/RetryPolicyStrategyTests.swift 100.00% <ø> (ø)
...ts/TyphoonTests/UnitTests/RetrySequenceTests.swift 100.00% <ø> (ø)

@ns-vasilev ns-vasilev merged commit c6199f8 into main Feb 22, 2026
39 checks passed
@ns-vasilev ns-vasilev deleted the refactor/strategies branch February 22, 2026 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants