Skip to content
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

Update README.md - Add monthly downloads badge #1607

Merged
merged 2 commits into from
May 24, 2024

Conversation

ayushjain01
Copy link
Contributor

Added a monthly downloads badge from pip Trends to the README.
View more at - https://piptrends.com/widgets/fsspec

@martindurant
Copy link
Member

How is this different from the "pypi downloads" immediately above?

@ayushjain01
Copy link
Contributor Author

The stats are slightly more accurate and refreshed daily, you can see the significant difference in the download counts between the two badges. Also there are more options in terms of badges and widgets to display stats about the package.
You can check them out here - https://piptrends.com/widgets/fsspec and https://piptrends.com/package/fsspec

@martindurant
Copy link
Member

So you would say this should replace the other badge? Having both seems silly.

@ayushjain01
Copy link
Contributor Author

yes it should

@martindurant martindurant merged commit c8112a2 into fsspec:master May 24, 2024
@ixmatus
Copy link

ixmatus commented Jun 27, 2024

@martindurant a colleague of this person tried to open this exact PR against a project I'm the maintainer for as well (ixmatus/inflector#16). This is a promotional campaign and they (or colleagues) appear to be doing it on reddit too, I think this is suspicious activity. They are creating an external dependency on a web property that (as far as I can tell) is not officially blessed by pypi. I strongly recommend reverting this change.

Users can get package statistics from libraries.io which is linked to directly by pypi itself.

martindurant added a commit to martindurant/filesystem_spec that referenced this pull request Jun 27, 2024
@martindurant
Copy link
Member

Thanks for the heads up, @ixmatus : #1636 . @ayushjain01 , you have until tomorrow to respond.

@ixmatus
Copy link

ixmatus commented Jun 27, 2024

@martindurant I appreciate you're giving them the benefit of the doubt and a chance to object but I thought you should know I went through quite a few python github projects and left this warning where they've shot gunned this exact change out.

The innocent interpretation is that it's promotional and SEO link juicing their site. The nefarious interpretation is that they've now acquired first-time contributor status to your repository so GitHub now gives them the "Contributor" label on any subsequent PRs they open (a little bit of far fetched social engineering attack vector but I work in security so I tend towards paranoid).

martindurant added a commit that referenced this pull request Jun 27, 2024
* Revert #1607

* Just remove it
@ayushjain01
Copy link
Contributor Author

Thank you for your feedback and concerns. I'd like to clarify that this is not a promotional campaign. As stated in the PR description, our intention is simply to share a tool we have developed, which we believe could be beneficial to the community.

Our badge system offers highly accurate statistics, which is why we wanted to make it available. If the link to the pip trends page is a concern, we are more than willing to remove it.

We respect your decision if you prefer not to add more badges to your README. Our primary goal is to address any misunderstandings and assure you that this is a genuine effort to contribute to the open-source community by sharing our work on GitHub.

Thank you for considering our contribution.

@martindurant
Copy link
Member

Thanks for replying @ayushjain01 . In fact, I ended up simply removing the download badge - fsspec doesn't need to advertise it's stats, and users can find the information for themselves if they wish.

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