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

docs: Center align README badges #2959

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

docs: Center align README badges #2959

wants to merge 4 commits into from

Conversation

kieran-ryan
Copy link
Member

🤔 What's changed?

  • Center align README badges
  • Change version badge color from blue to green
  • Fix GitHub workflow badge
  • Extended summary description and linked logo

Before

#StandWithUkraine OpenCollective OpenCollective Maven Central Build Status Coverage Status

After

Cucumber JVM version Build status Coverage Cucumber backers Cucumber sponsors Ukraine solidarity

⚡️ What's your motivation?

🏷️ What kind of change is this?

  • 📖 Documentation (improvements without changing code)

♻️ Anything particular you want feedback on?

  • "Automated tests in plain language, for JVM" tagline suitability
  • External permalink logo suitability - or whether to include a copy of the SVG in the repository alternatively
  • Whether the summary description is suitable or requires updates

📋 Checklist:

@kieran-ryan kieran-ryan added the 📖 documentation Improvements or additions to documentation label Dec 27, 2024
@kieran-ryan kieran-ryan self-assigned this Dec 27, 2024
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@mpkorstanje mpkorstanje left a comment

Choose a reason for hiding this comment

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

LGTM. Feel free to merge after addressing the nitpicks.

@kieran-ryan
Copy link
Member Author

kieran-ryan commented Dec 28, 2024

Appreciate the review @mpkorstanje - changes applied with both comments - thanks for flagging! I would be less push rights for main with this repo - being without need - so unable to merge my side, if suitable to apply merge

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.

2 participants