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

fix: use ./ syntax for exporting a package name export #61

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Feb 26, 2024

Trying to upgrade from 0.17.2 to 0.18.0 in a project already using esm and electron 28+ but I got errors while importing

import { aboutMenuItem } from 'electron-util/main';

was reporting Cannot find module 'electron-util/main' or its corresponding type declarations. error

adding the ./ prefix is allowing me to have the documented syntax working

else we got "Cannot find module 'electron-util/main' or its
corresponding type declarations." error
@sindresorhus sindresorhus merged commit 26822dc into sindresorhus:main Feb 26, 2024
3 checks passed
@benoitf benoitf deleted the named-export branch February 26, 2024 17:02
benoitf added a commit to benoitf/desktop that referenced this pull request Feb 26, 2024
for main import, need to use a suffix
also was requiring sindresorhus/electron-util#61

Signed-off-by: Florent Benoit <fbenoit@redhat.com>
benoitf added a commit to benoitf/desktop that referenced this pull request Feb 26, 2024
for main import, need to use a suffix
also was requiring sindresorhus/electron-util#61

Signed-off-by: Florent Benoit <fbenoit@redhat.com>
benoitf added a commit to benoitf/desktop that referenced this pull request Feb 26, 2024
for main import, need to use a suffix
also was requiring sindresorhus/electron-util#61

Signed-off-by: Florent Benoit <fbenoit@redhat.com>
benoitf added a commit to containers/podman-desktop that referenced this pull request Feb 28, 2024
for main import, need to use a suffix
also was requiring sindresorhus/electron-util#61

Signed-off-by: Florent Benoit <fbenoit@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants