Closed KatherineInCode closed 3 weeks ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 88.56%. Comparing base (
2ed93f0
) to head (d49c4b2
). Report is 6 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One â Scan Summary & Details â 2c29ca81-a464-469b-9042-bbf934300c1e
đī¸ Tracking
https://bitwarden.atlassian.net/browse/PM-11133
đ Objective
This is part of the iOS 18 / Xcode 16 / Swift 6 effort
This adds
Sendable
annotations to variousstruct
andenum
declarations that were not already implicitly Sendable. This greatly reduces the number of warnings with strict concurrency turned on. As well,@preconcurrency
annotations are being added toBitwardenSDK
imports because those objects will be marked as Sendable at some point in the future, and this enables the newly-Sendable objects to be warning-free.â° Reminders before review
đĻŽ Reviewer guidelines
:+1:
) or similar for great changes:memo:
) or âšī¸ (:information_source:
) for notes or general info:question:
) for questions:thinking:
) or đ (:thought_balloon:
) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion:art:
) for suggestions / improvements:x:
) or â ī¸ (:warning:
) for more significant problems or concerns needing attention:seedling:
) or âģī¸ (:recycle:
) for future improvements or indications of technical debt:pick:
) for minor or nitpick changes