Skip to content

chore: set up Knip#2835

Merged
tido64 merged 1 commit into
trunkfrom
tido/knip
Jul 1, 2026
Merged

chore: set up Knip#2835
tido64 merged 1 commit into
trunkfrom
tido/knip

Conversation

@tido64

@tido64 tido64 commented Jul 1, 2026

Copy link
Copy Markdown
Member

Description

Set up Knip and fix issues

Platforms affected

  • Android
  • iOS
  • macOS
  • visionOS
  • Windows

Test plan

CI should pass

@tido64 tido64 requested review from JasonVMo and Saadnajmi as code owners July 1, 2026 10:41
@github-actions github-actions Bot added platform: Android This affects Android platform: iOS This affects iOS platform: macOS This affects macOS platform: Windows This affects Windows platform: visionOS labels Jul 1, 2026
Comment on lines -252 to +255
../scripts/react-native.mts run-ios
../scripts/react-native.ts run-ios

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why did this change?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Because tools like oxfmt use the type field in package.json to determine whether files are ESM and complain if they're not. I've finally set root to be ESM so we don't need to use the .mjs/.mts extension any longer.

@tido64 tido64 merged commit b61e946 into trunk Jul 1, 2026
61 of 64 checks passed
@tido64 tido64 deleted the tido/knip branch July 1, 2026 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

platform: Android This affects Android platform: iOS This affects iOS platform: macOS This affects macOS platform: visionOS platform: Windows This affects Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants