Skip to content

fix: resolve ESLint errors #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
jaem0629 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: resolve ESLint errors #2

jaem0629 wants to merge 1 commit into from
+36 −33

Conversation

@jaem0629
Copy link

@jaem0629 jaem0629 commented Jan 23, 2026 •
edited
Loading

🎯 Changes

Fixed ESLint errors in package.

  • Fix import order
  • Change array types from T[] to Array<T>
  • Add explicit type assertion for Object.entries to preserve undefined possibility
  • Remove unnecessary optional chaining on navigator properties

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm run test:pr.

🚀 Release Impact

  • This change affects published code, and I have generated a changeset.
  • This change is docs/CI/dev-only (no release).

@jaem0629
fix: resolve ESLint errors in @tanstack/keys
422c1eb 
- Fix import order (type imports after regular imports)
- Change array types from T[] to Array<T> per project style
- Add explicit type assertion for Object.entries to preserve undefined
- Remove unnecessary optional chaining on navigator properties
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jaem0629