Skip to content

Conversation

@ns-vasilev
Copy link
Member

No description provided.

@ns-vasilev ns-vasilev self-assigned this Dec 19, 2025
@ns-vasilev ns-vasilev requested a review from a team as a code owner December 19, 2025 18:28
@spacecode-github-bot
Copy link

spacecode-github-bot commented Dec 19, 2025

1 Warning
⚠️

⚠️ Some source files are missing corresponding test files:

  • QueryParametersFormatter → expected test file: QueryParametersFormatterTests.swift
  • NetworkLayerAssembly → expected test file: NetworkLayerAssemblyTests.swift

Generated by 🚫 Danger

@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

❌ Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.27%. Comparing base (bc62114) to head (fedabba).
⚠️ Report is 5 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ParametersFormatter/QueryParametersFormatter.swift 20.00% 4 Missing ⚠️

❌ Your patch status has failed because the patch coverage (20.00%) is below the target coverage (85.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
+ Coverage   89.20%   89.27%   +0.07%     
==========================================
  Files          43       43              
  Lines        1472     1472              
==========================================
+ Hits         1313     1314       +1     
+ Misses        159      158       -1     
Files with missing lines Coverage Δ
.../Core/Builders/RequestBuilder/RequestBuilder.swift 100.00% <ø> (ø)
...NetworkLayer/Classes/DI/NetworkLayerAssembly.swift 0.00% <ø> (ø)
...lasses/Helpers/Helpers/RequestProcessor+Mock.swift 100.00% <ø> (ø)
.../Classes/Tests/UnitTests/RequestBuilderTests.swift 98.33% <ø> (ø)
...ParametersFormatter/QueryParametersFormatter.swift 23.08% <20.00%> (+1.92%) ⬆️

@ns-vasilev ns-vasilev merged commit 92df79f into main Dec 20, 2025
24 of 25 checks passed
@ns-vasilev ns-vasilev deleted the lint branch December 20, 2025 05:20
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.

3 participants