Skip to content

Conversation

@LitoMore
Copy link
Contributor

@LitoMore LitoMore commented Mar 5, 2025

See https://github.com/community/maintainers/discussions/442#discussioncomment-12401989

The npm-compare.com is a fake site of npmcompare.com. This user has been terminated, and the GitHub staff has deleted all npm-compare.com PRs.

@LitoMore LitoMore requested a review from a team as a code owner March 5, 2025 17:04
@netlify
Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for fakerjs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit c0c4084
🔍 Latest deploy log https://app.netlify.com/sites/fakerjs/deploys/67c88410ba454e000865a304
😎 Deploy Preview https://deploy-preview-3433.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 site configuration.

@Shinigami92
Copy link
Member

See community/maintainers/discussions/442#discussioncomment-12401989

The npm-compare.com is a fake site of npmcompare.com. This user has been terminated, and the GitHub staff has deleted all npm-compare.com PRs.

Your link (https://github.com/community/maintainers/discussions/442#discussioncomment-12401989) is leading to a 404 page?

@ST-DDT
Copy link
Member

ST-DDT commented Mar 5, 2025

The page npmcompare.com does not return a valid page.
npm-compare.com does still seem to work properly and is also listed as top result by google.

@ST-DDT
Copy link
Member

ST-DDT commented Mar 5, 2025

Previous PR: #3374
Affected commit: 597dc01
Deleted user: cwtuan

@LitoMore
Copy link
Contributor Author

LitoMore commented Mar 5, 2025

Your link (https://github.com/community/maintainers/discussions/442#discussioncomment-12401989) is leading to a 404 page?

You need to join the GitHub Maintainers private repository: https://maintainers.github.com.

See sindresorhus/awesome-npm@8ab222e. His behavior might be even worse. He also pretended to be the npmcomapre.com website and changed the links to npm-compare.com on many projects.

@Shinigami92 Shinigami92 added c: docs Improvements or additions to documentation p: 1-normal Nothing urgent c: security Indicates a vulnerability labels Mar 5, 2025
@Shinigami92 Shinigami92 added this to the vAnytime milestone Mar 5, 2025
@codecov
Copy link

codecov bot commented Mar 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.97%. Comparing base (bf3aa8b) to head (c0c4084).
Report is 1 commits behind head on next.

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #3433   +/-   ##
=======================================
  Coverage   99.97%   99.97%           
=======================================
  Files        2811     2811           
  Lines      216972   216972           
  Branches      944      944           
=======================================
+ Hits       216917   216919    +2     
+ Misses         55       53    -2     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@ST-DDT ST-DDT left a comment

Choose a reason for hiding this comment

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

Approved, but pending the validation of the claims.

@ST-DDT ST-DDT requested review from a team March 5, 2025 17:35
@ST-DDT
Copy link
Member

ST-DDT commented Mar 6, 2025

IMO the feature isn't worth spending more time for it to become clear whether this was overly enthusiasm or malice.

@ST-DDT ST-DDT merged commit 57d39d7 into faker-js:next Mar 6, 2025
26 checks passed
@LitoMore LitoMore deleted the revert-npm-download-badge branch March 6, 2025 17:01
@cwtuan
Copy link
Contributor

cwtuan commented Jul 21, 2025

Hi, I'm the creator of npm-compare.

I understand that my account was likely potentially fraudulent because of the numerous pull requests I sent to promote my website, npm-compare.com. I sincerely apologize for any inconvenience this might have caused.

My intention behind creating npm-compare was to offer a valuable tool to the developer community. Eager to share it globally, I made the error of over - promoting it through mass pull requests.

I now recognize my mistake, understanding that this behavior not only disrupted project maintainers but also violated GitHub's rules. I've learned from this experience. Going forward, I will strictly adhere to GitHub's guidelines and use only proper channels to promote my projects.

I have now deleted the promotional pull requests. I once again apologize for any disruption my actions may have caused.

I deeply appreciate the work you all do, and I don’t want to add to your burden.

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

Labels

c: docs Improvements or additions to documentation c: security Indicates a vulnerability p: 1-normal Nothing urgent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants