-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Build: Bundle lib/channel-postmessage with ts-up #19388
Build: Bundle lib/channel-postmessage with ts-up #19388
Conversation
Socket Security Report👍 No new dependency issues detected in pull request Socket.dev scan summary
Bot CommandsTo ignore an alert, reply with a comment starting with Powered by socket.dev |
@IanVS & @ndelangen do you guys know what should I do to pass the |
@javier-arango I think those are issues with the current state of the repo, and not something you've done here. |
@IanVS sounds good, should I put the PR ready for review ? |
…message-tsup # Conflicts: # code/lib/channel-postmessage/package.json
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.
Woo hoo, CI is green! Thanks @javier-arango!
@IanVS Finally we got green CI, I am going to merge the changes now. Thank you for the help. |
Issue: #18732
What I did
Migrated
lib/channel-postmessage
to use thets-up
bundler.How to test
If your answer is yes to any of these, please make sure to include it in your PR.