Skip to content

docs(readme): set the logo apart from the heading - #649

Merged
mambax7 merged 1 commit into
masterfrom
docs/readme-logo-spacing
Oct 14, 2026
Merged

mambax7 merged 1 commit into
masterfrom
docs/readme-logo-spacing

Conversation

@mambax7

@mambax7 mambax7 commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Blank lines before and after the logo image so it renders as its own paragraph instead of running into the # XOOPS Core heading. README only.

Summary by Sourcery

Documentation:

  • Improve README formatting by separating the XOOPS logo from the main heading.

Summary by CodeRabbit

  • Documentation
    • Improved spacing around the XOOPS CMS logo in the README.

Copilot AI balanced review requested due to automatic review settings October 14, 2026 06:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sourcery-ai

sourcery-ai Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

The README now uses blank-line spacing to render the XOOPS logo separately from the “XOOPS Core” heading, with no other content changes.

File-Level Changes

Change Details Files
Separated the repository logo from the main heading in the README’s Markdown structure.
  • Added blank lines before and after the logo image so it renders as a standalone paragraph.
  • Kept the change limited to README presentation formatting.
README.md

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@coderabbitai

coderabbitai Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: XOOPS/XoopsCore26/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Essentials
  • Run ID: fb64a73a-09f2-44ed-a24c-ece1be2b3956
📥 Commits

Reviewing files that changed from the base of the PR and between 7714c92 and 52bd4b1.

📒 Files selected for processing: 1
  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 5 included reviews per hour; 3 remain after this review.

Walkthrough

The README adds blank lines before and after the existing XOOPS CMS logo. The image and its URL are unchanged.

Changes

README formatting

Layer / File(s) Summary
Logo spacing
README.md
Blank lines are added before and after the existing XOOPS CMS logo. The image and its URL are unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 52bd4

This README spacing change preserves the existing logo and its link, with no identified impact on users. It is ready to merge after normal checks.

Pre-merge checks: Passed 5
Passed Passed checks: 5 passed
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the README logo-spacing change. It matches the primary change in the pull request.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@sonarqubecloud

Copy link
Copy Markdown

@gitar-bot

gitar-bot Bot commented Oct 14, 2026

Copy link
Copy Markdown

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

Qodana for PHP

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Detected 54 dependencies

Third-party software list

This page lists the third-party software dependencies used in project

Dependency Version Licenses
assetic/framework v3.2.3 MIT
bacon/bacon-qr-code v3.1.1 BSD-2-Clause
composer/ca-bundle 1.5.14 MIT
dasprid/enum 1.0.7 BSD-2-Clause
dflydev/apache-mime-types v1.0.1 MIT
doctrine/dbal 4.5.0 MIT
doctrine/deprecations 1.1.6 MIT
embed/embed v4.4.19 MIT
endroid/qr-code 6.1.3 MIT
ezyang/htmlpurifier v4.19.1 LGPL-2.1-or-later
firebase/php-jwt v7.2.1 BSD-3-Clause
geekwright/po v2.0.2 GPL-2.0-or-later
kint-php/kint 6.1.0 MIT
league/container 4.2.5 MIT
league/uri-interfaces 7.8.1 MIT
league/uri 7.8.1 MIT
matthiasmullie/minify 1.3.75 MIT
matthiasmullie/path-converter 1.1.3 MIT
ml/iri 1.1.4 MIT
ml/json-ld 1.2.1 MIT
moneyphp/money v4.9.0 MIT
monolog/monolog 3.12.1 MIT
oscarotero/html-parser v0.1.8 MIT
php-debugbar/php-debugbar v3.8.0 MIT
phpmailer/phpmailer v6.12.0 LGPL-2.1-only
psr/cache 3.0.0 MIT
psr/container 2.0.2 MIT
psr/http-client 1.0.3 MIT
psr/http-factory 1.1.0 MIT
psr/http-message 2.0 MIT
psr/log 3.0.2 MIT
ptachoire/cssembed v1.0.2 MIT
punic/punic 3.8.2 MIT
scssphp/scssphp v2.1.0 MIT
scssphp/source-span v1.1.0 MIT
smarty/smarty v4.5.8 LGPL-3.0-only
stefangabos/zebra_image 2.8.2 LGPL-3.0-only
symfony/console v7.4.20 MIT
symfony/deprecation-contracts v3.7.1 MIT
symfony/filesystem v8.0.15 MIT
symfony/polyfill-ctype v1.37.0 MIT
symfony/polyfill-intl-grapheme v1.43.0 MIT
symfony/polyfill-intl-normalizer v1.43.0 MIT
symfony/polyfill-mbstring v1.43.0 MIT
symfony/process v7.4.19 MIT
symfony/service-contracts v3.7.3 MIT
symfony/string v8.0.15 MIT
symfony/var-dumper v8.0.15 MIT
symfony/yaml v7.4.20 MIT
tecnickcom/tcpdf 6.11.4 LGPL-3.0-or-later
tedivm/stash v1.2.2 BSD-3-Clause
webmozart/assert 1.12.1 MIT
wikimedia/less.php v5.5.1 Apache-2.0
xoops/regdom v2.1.1 Apache-2.0
MPL-2.0
Contact Qodana team

Contact us at qodana-support@jetbrains.com

@mambax7
mambax7 merged commit 38b09db into master Oct 14, 2026
12 checks passed
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.

2 participants