Skip to content

Add typescript-type-guard logo to README.md - #7

Merged
Coderrob merged 1 commit into
mainfrom
copilot/add-typescript-type-guard-logo
Mar 31, 2026
Merged

Coderrob merged 1 commit into
mainfrom
copilot/add-typescript-type-guard-logo

Conversation

@Coderrob

Copy link
Copy Markdown
Owner

Copilot AI review requested due to automatic review settings March 31, 2026 23:24
@Coderrob
Coderrob merged commit fc9085e into main Mar 31, 2026
3 checks passed
@Coderrob
Coderrob deleted the copilot/add-typescript-type-guard-logo branch March 31, 2026 23:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a centered project logo image to the top of the README to improve repository/package presentation.

Changes:

  • Inserted an HTML block in README.md to render the logo image centered under the main heading.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
# typescript-type-guards

<p align="center">
<img src="public/img/typescript-type-guard-logo.png" alt="typescript-type-guards logo" />

Copilot AI Mar 31, 2026

Copy link

Choose a reason for hiding this comment

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

The image is referenced via a relative path (public/img/...). Since package.json only publishes dist in the files array, public/ won’t be included in the npm package and the logo will be broken on the npm README page. Consider switching src to an absolute GitHub raw/asset URL, or include public/img in the published package (update files in package.json) so the relative link can resolve.

Suggested change
<img src="public/img/typescript-type-guard-logo.png" alt="typescript-type-guards logo" />
<img src="https://raw.githubusercontent.com/OWNER/REPO/main/public/img/typescript-type-guard-logo.png" alt="typescript-type-guards logo" />

Copilot uses AI. Check for mistakes.
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.

3 participants