Skip to content

Remove unused GoogleFont story import#170

Open
thehanslevi wants to merge 1 commit into
oopsio:mainfrom
thehanslevi:codex/remove-unused-googlefont-story-import
Open

Remove unused GoogleFont story import#170
thehanslevi wants to merge 1 commit into
oopsio:mainfrom
thehanslevi:codex/remove-unused-googlefont-story-import

Conversation

@thehanslevi

@thehanslevi thehanslevi commented Jun 18, 2026

Copy link
Copy Markdown

Summary

  • remove the unused h import from the GoogleFont Storybook story
  • leave the story implementation unchanged

Refs #6

Validation

  • npx eslint packages/jen/src/stories/GoogleFont.stories.tsx
  • git diff --check

Note: I also ran broader source lint while validating the issue and it still reports unrelated pre-existing errors in other files. This PR only addresses the unused import flagged in the GoogleFont story.

Summary by CodeRabbit

  • Chores
    • Removed an unnecessary import dependency, optimizing the codebase with no impact to functionality or user experience.

@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the kessud2021's projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify

netlify Bot commented Jun 18, 2026

Copy link
Copy Markdown

Deploy Preview for polite-rabanadas-e7f4e1 failed.

Name Link
🔨 Latest commit f5b83db
🔍 Latest deploy log https://app.netlify.com/projects/polite-rabanadas-e7f4e1/deploys/6a340a052d33720008134d3f

@coderabbitai

coderabbitai Bot commented Jun 18, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9e065809-30a9-44b6-86a1-bd26f92743fb

📥 Commits

Reviewing files that changed from the base of the PR and between 4645f54 and f5b83db.

📒 Files selected for processing (1)
  • packages/jen/src/stories/GoogleFont.stories.tsx
💤 Files with no reviewable changes (1)
  • packages/jen/src/stories/GoogleFont.stories.tsx

📝 Walkthrough

Walkthrough

Removes the explicit import { h } from 'preact' statement from GoogleFont.stories.tsx. No story exports, component logic, or other imports are modified. The file now relies on the JSX transform provided by the Storybook/Vite Preact tooling.

Changes

Remove redundant Preact h import

Layer / File(s) Summary
Drop explicit h import from GoogleFont stories
packages/jen/src/stories/GoogleFont.stories.tsx
Deletes the import { h } from 'preact' line; all exported stories and component logic remain unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

A single line removed, so neat and so light,
The h from preact hops out of sight,
The tooling provides what the import once gave,
One less little line for the bunny to save! 🐇✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description provides a clear summary and validation steps, but does not follow the repository's required template structure with sections like Type of change, How Has This Been Tested, and Checklist. Update the description to follow the repository template, including sections for Type of change, testing approach, and the completion checklist to ensure consistency with project standards.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: removing an unused import from the GoogleFont story file. It is concise, specific, and directly related to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

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