Skip to content
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.

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

meta: use file extension in all file imports #3980

Closed
wants to merge 2 commits into from

Conversation

aduh95
Copy link
Member

@aduh95 aduh95 commented Aug 15, 2022

import/extensions seems to work fine for our use case, the only problem is that it doesn't support exports map, fortunately that seems to be a problem only for nanoid.

@aduh95 aduh95 requested a review from arturi August 15, 2022 14:48
@aduh95 aduh95 added safe to test Add this label on trustworthy PRs to spawn the e2e test suite and removed pending end-to-end tests labels Aug 15, 2022
@github-actions github-actions bot removed the safe to test Add this label on trustworthy PRs to spawn the e2e test suite label Aug 15, 2022
@aduh95 aduh95 added the safe to test Add this label on trustworthy PRs to spawn the e2e test suite label Aug 15, 2022
@github-actions github-actions bot removed the safe to test Add this label on trustworthy PRs to spawn the e2e test suite label Aug 15, 2022
@aduh95
Copy link
Member Author

aduh95 commented Aug 15, 2022

Not sure what to do with packages/@uppy/utils/types/index.d.ts (currently it uses no extensions).

@arturi
Copy link
Contributor

arturi commented Aug 15, 2022

Not sure what to do with packages/@uppy/utils/types/index.d.ts (currently it uses no extensions).

Why not add them there?

@aduh95 aduh95 closed this Aug 16, 2022
@aduh95 aduh95 deleted the file-extension-imports branch August 16, 2022 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants