Skip to content

chore: Update ppom package#9342

Merged
pedronfigueiredo merged 2 commits intomainfrom
ppom_update_1.4.6
Apr 22, 2024
Merged

chore: Update ppom package#9342
pedronfigueiredo merged 2 commits intomainfrom
ppom_update_1.4.6

Conversation

@pedronfigueiredo
Copy link
Copy Markdown
Contributor

@pedronfigueiredo pedronfigueiredo commented Apr 22, 2024

Description

Update @blockaid/ppom_release package to 1.4.5

Related issues

Fixes: #24172

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

  • I’ve followed MetaMask Coding Standards.
  • I've completed the PR template to the best of my ability
  • I’ve included tests if applicable
  • I’ve documented my code using JSDoc format if applicable
  • I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@pedronfigueiredo pedronfigueiredo added team-confirmations-secure-ux-deprecated DEPRECATED: please use "team-confirmations" instead team-confirmations-secure-ux-PR PR from the confirmations team labels Apr 22, 2024
@pedronfigueiredo pedronfigueiredo self-assigned this Apr 22, 2024
@pedronfigueiredo pedronfigueiredo requested a review from a team as a code owner April 22, 2024 14:00
@pedronfigueiredo pedronfigueiredo requested a review from a team April 22, 2024 14:00
@pedronfigueiredo pedronfigueiredo requested a review from a team as a code owner April 22, 2024 14:00
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 22, 2024 •
edited
Loading

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.

@socket-security
Copy link
Copy Markdown

socket-security bot commented Apr 22, 2024 •
edited
Loading

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@blockaid/ppom_release@1.4.6 network 0 3.32 MB blockaiduser

🚮 Removed packages: npm/@blockaid/ppom_release@1.4.5

View full report↗︎

jpuri
jpuri previously approved these changes Apr 22, 2024
View reviewed changes
@pedronfigueiredo
Copy link
Copy Markdown
Contributor Author

pedronfigueiredo commented Apr 22, 2024

I have read the CLA Document and I hereby sign the CLA

@pedronfigueiredo pedronfigueiredo force-pushed the ppom_update_1.4.6 branch 2 times, most recently from 5109557 to e6afb96 Compare April 22, 2024 16:39
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 22, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@pedronfigueiredo pedronfigueiredo merged commit 1e487f8 into main Apr 22, 2024
@pedronfigueiredo pedronfigueiredo deleted the ppom_update_1.4.6 branch April 22, 2024 17:39
@github-actions github-actions bot locked and limited conversation to collaborators Apr 22, 2024
@metamaskbot metamaskbot added release-7.22.0 Issue or pull request that will be included in release 7.22.0 release-7.21.0 Issue or pull request that will be included in release 7.21.0 and removed release-7.22.0 Issue or pull request that will be included in release 7.22.0 labels Apr 22, 2024
@metamaskbot
Copy link
Copy Markdown
Collaborator

metamaskbot commented Apr 23, 2024

Missing release label release-7.21.0 on PR. Adding release label release-7.21.0 on PR and removing other release labels(release-7.22.0), as PR was cherry-picked in branch 7.21.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jpuri jpuri jpuri approved these changes

@cryptotavares cryptotavares cryptotavares approved these changes

+1 more reviewer

@digiwand digiwand digiwand approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@pedronfigueiredo pedronfigueiredo

Labels

release-7.21.0 Issue or pull request that will be included in release 7.21.0 team-confirmations-secure-ux-deprecated DEPRECATED: please use "team-confirmations" instead team-confirmations-secure-ux-PR PR from the confirmations team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@pedronfigueiredo @metamaskbot @jpuri @cryptotavares @digiwand