Conversation
|
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Refactors the translation utility import list and bumps the package version.
Changes:
- Removed the
tnamed import fromi18nextin the translation utility. - Bumped
package.jsonversion from1.7.0to1.7.1.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/utils/translation.ts | Adjusts i18next imports, removing t. |
| package.json | Increments package patch version. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| { | ||
| "name": "@stelliajs/framework", | ||
| "version": "1.7.0", | ||
| "version": "1.7.1", |
There was a problem hiding this comment.
The PR title indicates only removing an unused import, but this PR also bumps the package version. Either update the PR title/description to mention the version bump, or move the version bump to a separate PR to keep changes scoped.
| @@ -1,4 +1,4 @@ | |||
| import { i18n, t } from "i18next"; | |||
| import { i18n } from "i18next"; | |||
There was a problem hiding this comment.
If i18n is only used as a TypeScript type in this file, prefer a type-only import (import type { i18n } from \"i18next\";) to avoid emitting a runtime import. If it is used at runtime, double-check that i18next actually provides a runtime named export i18n in your build setup (otherwise import the correct runtime value, e.g., the default export).
| import { i18n } from "i18next"; | |
| import type { i18n } from "i18next"; |
No description provided.