Skip to content

Conversation

@ViKing-py
Copy link

No description provided.

@ViKing-py ViKing-py requested a review from a team as a code owner January 8, 2026 14:33
@netlify
Copy link

netlify bot commented Jan 8, 2026

Deploy Preview for fakerjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 66e3710
🔍 Latest deploy log https://app.netlify.com/projects/fakerjs/deploys/695fc02995c44d0008f01f5e
😎 Deploy Preview https://deploy-preview-3693.fakerjs.dev
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@xDivisionByZerox
Copy link
Member

What exactly is the improvement here? You simply rewrote the existing code using slightly different syntax - the overall use-case stays identical. I would have expected an analysis + suggestion in the PR description if you provide an "improvement" without a prior issue. Please provide this information otherwise I'm going to have to close your PR.

Furthermore, you already provided a PR in which you suggest changing the icon (FFR see #3692). This does not need to be in this PR as well.

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.

2 participants