Skip to content

Add: README badges#432

Merged
MichaReiser merged 3 commits intoastral-sh:mainfrom
esadek:badges
May 17, 2025
Merged

Add: README badges#432
MichaReiser merged 3 commits intoastral-sh:mainfrom
esadek:badges

Conversation

@esadek
Copy link
Contributor

@esadek esadek commented May 17, 2025

Add badges to the README. Same badges as those used in the READMEs of uv and ruff.

@esadek
Copy link
Contributor Author

esadek commented May 17, 2025

SCR-20250517-miwo

@zanieb
Copy link
Member

zanieb commented May 17, 2025

(I kind of don't like README badges, curious what others think — I was looking at removing most of them in astral-sh/uv#11257)

@dhruvmanila
Copy link
Member

I don't have strong opinions but if we do keep it, PyPI and Discord seems enough as I think others are present in the GitHub UI.

@MichaReiser
Copy link
Member

I'm fine with badges that serve a purpose and don't add redundant information. I agree with @dhruvmanila that I'd keep PyPI, Discord. The license is already visible on PyPI and GitHub and so is the CI status (which is rather useless on PyPI).

@esadek
Copy link
Contributor Author

esadek commented May 17, 2025

Makes sense. Removed the license and CI badges.

@MichaReiser MichaReiser added the documentation Improvements or additions to documentation label May 17, 2025
@MichaReiser MichaReiser enabled auto-merge (squash) May 17, 2025 16:34
@MichaReiser MichaReiser merged commit 4ad13f2 into astral-sh:main May 17, 2025
10 checks passed
@esadek esadek deleted the badges branch May 17, 2025 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants