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: Export types from main and renderer, fix main types #58

Merged
merged 1 commit into from
Jun 2, 2021
Merged

fix: Export types from main and renderer, fix main types #58

merged 1 commit into from
Jun 2, 2021

Conversation

ffflorian
Copy link
Contributor

@ffflorian ffflorian commented May 17, 2021

This fixes #52.
This fixes #60.

cc @nornagon

tsconfig.json Outdated Show resolved Hide resolved
@ffflorian
Copy link
Contributor Author

@nornagon can you review again? :)

@nornagon nornagon merged commit 1502a4c into electron:master Jun 2, 2021
andersk added a commit to andersk/electron-remote that referenced this pull request Mar 1, 2022
This was previously excluded (electron#48) because of CI failures due to
index.d.ts referencing files in dist.  Those references were later
removed (electron#58). so we can include it again to have CI verify that
index.d.ts type checks.

Signed-off-by: Anders Kaseorg <andersk@mit.edu>
andersk added a commit to andersk/electron-remote that referenced this pull request Mar 1, 2022
This was previously excluded (electron#48) because of CI failures due to
index.d.ts referencing files in dist.  Those references were later
removed (electron#58), so we can include it again to have CI verify that
index.d.ts type checks.

Signed-off-by: Anders Kaseorg <andersk@mit.edu>
nornagon pushed a commit that referenced this pull request Mar 15, 2022
* fix: types for nativeImage, webContents, webFrameMain again

Fixes #110.

Signed-off-by: Anders Kaseorg <andersk@mit.edu>

* ci: Remove Electron 11

ShareMenu and webFrameMain are new in Electron 12.

Signed-off-by: Anders Kaseorg <andersk@mit.edu>

* fix: Remove exclusion of index.d.ts from tsconfig

This was previously excluded (#48) because of CI failures due to
index.d.ts referencing files in dist.  Those references were later
removed (#58), so we can include it again to have CI verify that
index.d.ts type checks.

Signed-off-by: Anders Kaseorg <andersk@mit.edu>
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.

Type error with @types/node 14.14.31 Could not find a declaration file for module @electron/remote/main
2 participants