-
-
Notifications
You must be signed in to change notification settings - Fork 760
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
Various SAST fixes #7644
Various SAST fixes #7644
Conversation
…more-random-fixes
✅ Deploy Preview for inventree-web-pui-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7644 +/- ##
==========================================
- Coverage 83.75% 83.74% -0.02%
==========================================
Files 1102 1102
Lines 48694 48696 +2
Branches 1550 1551 +1
==========================================
- Hits 40786 40782 -4
- Misses 7511 7516 +5
- Partials 397 398 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@SchrodingersGat found the CI problem |
@matmair what was the issue? |
@SchrodingersGat The StylishText component causes errors when getting empty children passed in so I reveres the change in that area |
Nice :) Happy for me to merge this then? |
@SchrodingersGat yes, ready for merge |
More small reliability/maintainability fixes from SAST warnings.