Skip to content

chore: enforce file extensions in relative imports - #770

Open
B4nan wants to merge 1 commit into
masterfrom
chore/lint-import-extensions
Open

B4nan wants to merge 1 commit into
masterfrom
chore/lint-import-extensions

Conversation

@B4nan

@B4nan B4nan commented Oct 9, 2026

Copy link
Copy Markdown
Member

Enables oxlint's import/extensions rule, so a relative import without the .js extension fails lint. src is already covered by NodeNext resolution, but test/tsconfig.json uses Bundler and lets these through (see #759 (comment)). Package imports are ignored, and master passes with the rule on.

@B4nan B4nan added adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. labels Oct 9, 2026
@B4nan
B4nan requested a review from janbuchar October 9, 2026 18:04
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown

⚠️ There are broken links in the documentation.

See more at https://github.com/apify/apify-sdk-js/actions/runs/37970513631#summary-113955621152

@B4nan
B4nan enabled auto-merge (squash) October 9, 2026 18:05
l2ysho pushed a commit to apify/apify-cli that referenced this pull request Oct 9, 2026
Enables oxlint's `import/extensions` rule, so a relative import without
the `.js` extension fails lint. Package imports are ignored, and the
codebase already passes. Same rule as apify/apify-sdk-js#770.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant