Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโ€™ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PM-15377: Rolled back review prompt legacy api #1218

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

fix lint warning

0918818
Select commit
Loading
Failed to load commit list.
Open

PM-15377: Rolled back review prompt legacy api #1218

fix lint warning
0918818
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 18, 2024 in 1s

41.02% of diff hit (target 89.43%)

View this Pull Request on Codecov

41.02% of diff hit (target 89.43%)

Annotations

Check warning on line 22 in BitwardenShared/UI/Platform/Application/Appearance/Modifiers/ReviewModifier.swift

See this annotation in the file changed.

@codecov codecov / codecov/patch

BitwardenShared/UI/Platform/Application/Appearance/Modifiers/ReviewModifier.swift#L20-L22

Added lines #L20 - L22 were not covered by tests

Check warning on line 43 in BitwardenShared/UI/Platform/Application/Appearance/Modifiers/ReviewModifier.swift

See this annotation in the file changed.

@codecov codecov / codecov/patch

BitwardenShared/UI/Platform/Application/Appearance/Modifiers/ReviewModifier.swift#L35-L43

Added lines #L35 - L43 were not covered by tests

Check warning on line 363 in BitwardenShared/UI/Vault/Vault/VaultList/VaultListView.swift

See this annotation in the file changed.

@codecov codecov / codecov/patch

BitwardenShared/UI/Vault/Vault/VaultList/VaultListView.swift#L362-L363

Added lines #L362 - L363 were not covered by tests

Check warning on line 375 in BitwardenShared/UI/Vault/Vault/VaultList/VaultListView.swift

See this annotation in the file changed.

@codecov codecov / codecov/patch

BitwardenShared/UI/Vault/Vault/VaultList/VaultListView.swift#L367-L375

Added lines #L367 - L375 were not covered by tests