Skip to content

Fix electron target #3

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

Merged
merged 9 commits into from
Mar 2, 2020
Merged

Fix electron target #3

merged 9 commits into from
Mar 2, 2020

Conversation

imsobear
Copy link
Owner

@imsobear imsobear commented Mar 1, 2020

No description provided.

@@ -10,7 +10,8 @@ function createWindow(): BrowserWindow {
width: 800,
height: 600,
webPreferences: {
preload: path.join(__dirname, 'preload.js')
preload: path.join(__dirname, 'preload.js'),
nodeIntegration: true
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"rootDir": "./",
"outDir": "../main_dist"
}
}
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

跟前端的 tsconfig 要分开下

@imsobear
Copy link
Owner Author

imsobear commented Mar 1, 2020

有个问题可能要处理下:main 的代码变更后必须重新 app:start 才能生效?

@imsobear
Copy link
Owner Author

imsobear commented Mar 1, 2020

还有一个:

image

我看下怎么在项目内能统一定义 registry,另外已有的怎么才能改掉?

@imsobear
Copy link
Owner Author

imsobear commented Mar 1, 2020

还有一个:

image

我看下怎么在项目内能统一定义 registry,另外已有的怎么才能改掉?

这个问题我先用编辑器统一改掉了,这个问题主要还是 yarn 不应该在 lock 文件里冗余 registry,我看这个 feature 可能要到 2.0 才能加:yarnpkg/yarn#5892

暂时的话:如果全局没配置自定义的 registry 那就没问题,如果配置了话,每次添加依赖时加一下命令行参数:--registry https://registry.yarnpkg.com

@codecov
Copy link

codecov bot commented Mar 1, 2020

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master     #3   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines           5      5           
=====================================
  Hits            5      5

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4671ce4...510bf9a. Read the comment docs.

Copy link
Collaborator

@noyobo noyobo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@noyobo noyobo merged commit 84a8352 into master Mar 2, 2020
@noyobo noyobo deleted the fix-electron-target branch March 2, 2020 02:15
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.

2 participants