Skip to content

docs(readme): show the Tabularis and BigQuery logos with release badges - #4

Merged
debba merged 1 commit into
mainfrom
docs/readme-logos
Oct 18, 2026
Merged

debba merged 1 commit into
mainfrom
docs/readme-logos

Conversation

@debba

@debba debba commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

Matches the header of the other driver READMEs: Tabularis logo, a plus sign and the BigQuery logo (bigquery-icon.svg, the same icon the manifest uses), followed by the Release, Downloads and CI badges.

@debba
debba merged commit 99d24bf into main Oct 18, 2026
7 checks passed
Comment thread README.md

# tabularis-bigquery-plugin

<p align="center">

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SUGGESTION: <p align="center"> will not center these badges as written

The blank lines inside the <p> end the CommonMark HTML block, so the Markdown badges below are rendered in their own separate plain <p>. In HTML5 parsing, that new <p> start tag implicitly closes the open <p align="center">, so align="center" ends up on an empty paragraph and the badges render left-aligned. Use <div align="center"> … </div> instead (a <div> stays open across the rendered paragraph), or put the badges inside the <p> as raw HTML <img>/<a> tags with no blank lines.


Reply with @kilocode-bot fix it to have Kilo Code address this issue.

@kilo-code-bot

kilo-code-bot Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 1
Issue Details (click to expand)

SUGGESTION

File Line Issue
README.md 9 <p align="center"> won't center the badges: blank lines end the HTML block and the badges render in a separate plain <p>, which implicitly closes the aligned one. Use <div align="center"> … </div> or raw HTML <img>/<a> tags without blank lines.
Files Reviewed (2 files)
  • README.md - 1 issue
  • assets/plus.svg - 0 issues

Verified: assets/plus.svg is added by this PR, bigquery-icon.svg exists at the repo root, and the CI workflow badge matches .github/workflows/ci.yml (name: CI).

Fix these issues in Kilo Cloud


Reviewed by kimi-k3 · Input: 176.1K · Output: 8.7K · Cached: 50.7K

@debba
debba deleted the docs/readme-logos branch October 18, 2026 15:10
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