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

Chore: contents read permission for unit tests #4268

Merged
merged 1 commit into from
Sep 27, 2024
Merged

Chore: contents read permission for unit tests #4268

merged 1 commit into from
Sep 27, 2024

Conversation

katspaugh
Copy link
Member

What it solves

Checking if we can downgrade the contents permission.

Copy link

github-actions bot commented Sep 27, 2024

Copy link

github-actions bot commented Sep 27, 2024

📦 Next.js Bundle Analysis for safe-wallet-web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

Copy link

github-actions bot commented Sep 27, 2024

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements
78.1% (+0.01% 🔼)
12000/15364
🔴 Branches
58.23% (+0.02% 🔼)
3085/5298
🟡 Functions 65.09% 1889/2902
🟡 Lines
79.65% (+0.01% 🔼)
10831/13599

Test suite run success

1484 tests passing in 203 suites.

Report generated by 🧪jest coverage report action from 0cfa831

@katspaugh katspaugh merged commit 74e2e42 into dev Sep 27, 2024
26 of 27 checks passed
@katspaugh katspaugh deleted the tests branch September 27, 2024 14:12
@github-actions github-actions bot locked and limited conversation to collaborators Sep 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants