Skip to content

I seem to have merged into the wrong branch#2890

Merged
inlined merged 23 commits into
nextfrom
master
Jun 23, 2026
Merged

I seem to have merged into the wrong branch#2890
inlined merged 23 commits into
nextfrom
master

Conversation

@inlined

@inlined inlined commented Jun 23, 2026

Copy link
Copy Markdown
Member

I guess next is the real branch?

CorieW and others added 23 commits May 18, 2026 14:59
Security Audit & Remediation: firestore-counter
…images

Security Audit & Remediation: storage-resize-images
…late-text

Security Audit & Remediation: firestore-translate-text
…d-nodes

Security Audit & Remediation: rtdb-limit-child-nodes
…en-urls-bitly

Security Audit & Remediation: firestore-shorten-urls-bitly
…email

Security Audit & Remediation: firestore-send-email
Security Audit & Remediation: delete-user-data
@inlined inlined requested a review from a team as a code owner June 23, 2026 17:59
@inlined inlined merged commit 3c04e6e into next Jun 23, 2026
15 checks passed
@wiz-9635d3485b

Copy link
Copy Markdown

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities 1 Medium
Data Finding Sensitive Data -
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations -
SAST Finding SAST Findings -
Software Management Finding Software Management Findings -
Total 1 Medium

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes several external dependencies, including rimraf, uuid, uuidv4, mkdirp, deep-equal, and node-fetch, across multiple Firebase extension functions. These are replaced with native Node.js APIs such as fs.rmSync, fs.promises.mkdir, crypto.randomUUID, and util.isDeepStrictEqual. Build scripts, package files, and lockfiles are updated accordingly. I have no feedback to provide on these changes.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants