-
Notifications
You must be signed in to change notification settings - Fork 897
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
Add tsec script to all packages #8285
Conversation
|
Changeset File Check
|
Size Report 1Affected ProductsNo changes between base commit (872e6f6) and merge commit (763c4a2).Test Logs |
Size Analysis Report 1Affected ProductsNo changes between base commit (872e6f6) and merge commit (763c4a2).Test Logs |
As a tangent we have to figure out why this keeps happening, it should no-op in this case. The changeset bot is already handling the "no changeset" case. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG, once we get the errors fixed we should add this to CI - not sure if it should have its own workflow or be grouped with something else. How long does it take to run?
Added the
yarn trusted-type-check
script to all packages. The script runs tsec to report possible security issues.Here is a sample output from the script, showing all warnings that we have to fix using safevalues: (internal) https://paste.googleplex.com/6754875962818560