Skip to content

docs: Fix links in README!#7723

Merged
JohnMcLear merged 1 commit into
developfrom
readme-tidy
May 11, 2026
Merged

docs: Fix links in README!#7723
JohnMcLear merged 1 commit into
developfrom
readme-tidy

Conversation

@JohnMcLear
Copy link
Copy Markdown
Member

Just some README polish to fix broken links..

@qodo-code-review
Copy link
Copy Markdown

ⓘ You've reached your Qodo monthly free-tier limit. Reviews pause until next month — upgrade your plan to continue now, or link your paid account if you already have one.

@qodo-free-for-open-source-projects
Copy link
Copy Markdown

Review Summary by Qodo

Fix broken links and update badges in README

📝 Documentation

Grey Divider

Walkthroughs

Description
• Remove outdated Sauce Labs badge references
• Simplify frontend test badge labels
• Fix broken Etherpad plugins link to correct URL
• Remove Windows Build badge from status section
Diagram
flowchart LR
  A["README.md badges section"] -->|Remove Sauce Labs refs| B["Simplified badges"]
  A -->|Update plugin link| C["Correct plugin URL"]
  B --> D["Cleaner status display"]
  C --> D
Loading

Grey Divider

File Changes

1. README.md 📝 Documentation +3/-5

Update badges and fix plugin documentation link

• Removed "powered by Sauce Labs" text from frontend test badges
• Removed Sauce Test Status badge and Windows Build badge
• Updated Etherpad plugins badge link from https://static.etherpad.org/index.html to
 https://etherpad.org/plugins
• Simplified badge labels for better clarity

README.md


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects
Copy link
Copy Markdown

qodo-free-for-open-source-projects Bot commented May 11, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (1)

Grey Divider


Remediation recommended

1. Explicit protocol in README URL 📘 Rule violation ⚙ Maintainability
Description
The newly added README link uses an explicit https:// URL, but the style rule requires
protocol-independent URLs (//...) where applicable. This can cause inconsistent URL style and
potential issues when rendered under different schemes.
Code

README.md[46]

+[![Etherpad plugins](https://img.shields.io/endpoint?url=https%3A%2F%2Fstatic.etherpad.org%2Fshields.json&color=%2344b492 "Etherpad plugins")](https://etherpad.org/plugins)
Evidence
PR Compliance ID 7 requires protocol-independent URLs (//...) where URLs are used in that context.
The added line in README.md includes explicit https:// URLs (both the badge image and the link
target).

README.md[46-46]
Best Practice: Repository guidelines

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
A newly added README link uses an explicit protocol (`https://`) but the style requirement calls for protocol-independent URLs (`//...`) where appropriate.

## Issue Context
This PR modifies `README.md` and introduces/keeps explicit-protocol URLs on the changed line.

## Fix Focus Areas
- README.md[46-46]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

@JohnMcLear JohnMcLear merged commit f784d2b into develop May 11, 2026
36 of 41 checks passed
@JohnMcLear JohnMcLear deleted the readme-tidy branch May 11, 2026 13:54
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.

1 participant