Skip to content

fix: add hashable conformance to authenticator states - #105

Draft
thisisabhash wants to merge 1 commit into
mainfrom
fix/hashable-warning
Draft

fix: add hashable conformance to authenticator states#105
thisisabhash wants to merge 1 commit into
mainfrom
fix/hashable-warning

Conversation

@thisisabhash

Copy link
Copy Markdown
Member

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov

codecov Bot commented Feb 11, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.34%. Comparing base (fd86328) to head (16ff6ba).
⚠️ Report is 8 commits behind head on main.

Files with missing lines Patch % Lines
.../Authenticator/States/AuthenticatorBaseState.swift 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
- Coverage   44.39%   44.34%   -0.06%     
==========================================
  Files          44       44              
  Lines        2462     2465       +3     
==========================================
  Hits         1093     1093              
- Misses       1369     1372       +3     
Flag Coverage Δ
Authenticator 44.34% <0.00%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

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.

1 participant