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
chore: move native-image.ts back into common #30838
Conversation
654c0ce
to
c4f2aeb
Compare
Release Notes Persisted
|
@@ -1,6 +1,7 @@ | |||
// Common modules, please sort alphabetically | |||
export const commonModuleList: ElectronInternal.ModuleEntry[] = [ | |||
{ name: 'clipboard', loader: () => require('./clipboard') }, | |||
{ name: 'nativeImage', loader: () => require('./native-image') }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicate of #
@@ -13,7 +13,7 @@ export const moduleList: ElectronInternal.ModuleEntry[] = [ | |||
}, | |||
{ | |||
name: 'nativeImage', | |||
loader: () => require('@electron/internal/renderer/api/native-image') | |||
loader: () => require('@electron/internal/common/api/native-image') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1,6 +1,7 @@ | |||
// Common modules, please sort alphabetically | |||
export const commonModuleList: ElectronInternal.ModuleEntry[] = [ | |||
{ name: 'clipboard', loader: () => require('./clipboard') }, | |||
{ name: 'nativeImage', loader: () => require('./native-image') }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- ``
@@ -13,7 +13,7 @@ export const moduleList: ElectronInternal.ModuleEntry[] = [ | |||
}, | |||
{ | |||
name: 'nativeImage', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -13,7 +13,7 @@ export const moduleList: ElectronInternal.ModuleEntry[] = [ | |||
}, | |||
{ | |||
name: 'nativeImage', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1,3 +0,0 @@ | |||
const { nativeImage } = process._linkedBinding('electron_common_native_image'); | |||
|
|||
export default nativeImage; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description of Change
It was split to separate browser and renderer implementation in #24802, which got undone in #30728 and we can merge them back into common.
Checklist
npm test
passesRelease Notes
Notes: notes