-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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: Move middleware into its own directory, add Wallet-API-Platform team to CODEOWNERS #26549
base: develop
Are you sure you want to change the base?
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
7dcbe95
to
33c7df6
Compare
Quality Gate failedFailed conditions |
Builds ready [9e7c8b2]
Page Load Metrics (80 ± 18 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #26549 +/- ##
===========================================
+ Coverage 69.96% 70.01% +0.05%
===========================================
Files 1405 1409 +4
Lines 49005 49115 +110
Branches 13703 13737 +34
===========================================
+ Hits 34284 34386 +102
- Misses 14721 14729 +8 ☔ View full report in Codecov by Sentry. |
Sonarcloud security issue is not introduced in this PR. Just activated in a file that's been moved. Just doesn't like use of |
This PR has been automatically marked as stale because it has not had recent activity in the last 60 days. It will be closed in 14 days. Thank you for your contributions. |
Description
app/scripts/lib
into its ownmiddleware
subdirectory,CODEOWNERS.md
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist