Skip to content

[WIP] Suffix all imports with .js #1491

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
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[WIP] Suffix all imports with .js #1491

wants to merge 1 commit into from

Conversation

Copilot
Copy link

@Copilot Copilot AI commented May 23, 2025 •
edited
Loading

  • Identify all packages that need to be updated to add the .js suffix to imports
  • Create a script to scan and update imports in all .ts files
  • Test the script on a small set of files to verify it works correctly
  • Run the script on all packages
  • Verify build works with npx tsc
  • Run tests to ensure everything functions correctly after the changes

Fixes #1001.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@julien-c
Copy link
Member

sounds good, let's go!

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

@julien-c julien-c

Copilot code review Copilot

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Suffix all imports with .js
2 participants
@Copilot @julien-c