Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@gwynne
Copy link
Member

@gwynne gwynne commented Dec 12, 2025

These changes are now available in 4.120.0

swift-log introduced CustomStringConvertible and LosslessStringConvertible conformances on Logger.Level which cause build errors by conflicting with our implementations, so ours are now gone.

…Convertible conformances to Logger.Level which cause build errors by conflicting with our implementations. Remove ours.
@gwynne gwynne requested review from 0xTim, MahdiBM and ptoffy December 12, 2025 21:33
@gwynne gwynne added the semver-minor Contains new API label Dec 12, 2025
@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.95%. Comparing base (8d47c18) to head (615f2c3).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3412      +/-   ##
==========================================
- Coverage   76.96%   76.95%   -0.01%     
==========================================
  Files         227      227              
  Lines        8699     8697       -2     
==========================================
- Hits         6695     6693       -2     
  Misses       2004     2004              
Files with missing lines Coverage Δ
...rces/Vapor/Logging/LoggingSystem+Environment.swift 100.00% <ø> (+5.00%) ⬆️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gwynne gwynne merged commit f7090db into main Dec 12, 2025
16 of 19 checks passed
@gwynne gwynne deleted the fix-swift-log-usage branch December 12, 2025 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver-minor Contains new API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants