Skip to content

Fix formatting by removing extra line break#196

Merged
ishaanxgupta merged 1 commit into
mainfrom
Ankit-Kotnala-patch-1
May 22, 2026
Merged

Fix formatting by removing extra line break#196
ishaanxgupta merged 1 commit into
mainfrom
Ankit-Kotnala-patch-1

Conversation

@Ankit-Kotnala
Copy link
Copy Markdown
Collaborator

Removed an extra line break in the README.

Removed an extra line break in the README.
@github-actions
Copy link
Copy Markdown

Fails
🚫

📋 PR description is too short. Please describe:

  • What changed and Why
  • Any relevant issue links (Closes #NNN)
  • Steps to test manually
Messages
📖

📝 No CHANGELOG.md update detected. If this PR introduces a user-visible change, please add an entry.

📖

✅ Targeting main. Please squash commits before merging to keep the git history clean.

Generated by 🚫 dangerJS against 64ead9b

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request modifies the README.md file by relocating a line break tag within the badges section. A review comment suggests that adding a hardcoded
tag can disrupt the layout across different screen sizes and recommends allowing the badges to flow naturally for better responsiveness.

Comment thread README.md
<img src="https://img.shields.io/badge/python-3.11+-blue?logo=python&logoColor=white" alt="Python 3.11+"/>
<img src="https://img.shields.io/badge/license-BSD--3--Clause-green" alt="BSD-3 License"/>
<img src="https://img.shields.io/badge/FastAPI-00C7B7?logo=fastapi&logoColor=white" alt="FastAPI"/>
<br/>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The addition of a hardcoded <br/> tag here forces a line break that can disrupt the layout on different screen sizes. For instance, on mobile devices, the badges may wrap naturally, and on wider screens, this forced break creates an unnecessary split. It is better to allow the badges to flow naturally within the centered container. Since the other badges are already on separate lines in the source, they will be rendered with appropriate spacing automatically.

@ishaanxgupta ishaanxgupta merged commit d1090f2 into main May 22, 2026
6 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants