-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
upgrade: 清理和升级依赖 #396
upgrade: 清理和升级依赖 #396
Conversation
我先把这个PR转成 draft 了,你好了以后,转成 open ,我再过来测试下。 |
原来 |
只在dev的时候依赖 |
已经好了吗?那我晚上去测试下。 ^_^ |
@starknt 能配置 |
if(c) {
// do something
} else { // do something ? 这样? } |
这样: if (c) {
// do something
} else {
// do something
} 而不是: if (c) {
// do something
}
else {
// do something
} |
就是 |
应该可以配置 |
|
没有安装 |
噢,好吧。哈哈。 |
问题2:dns.lookup 失败:
详细运行日志: |
这个 |
是啊,好几次这东西报错了。 |
现在好像没问题了,我再自己 PC 安装测试一下看看。 |
# Conflicts: # README.md # packages/cli/src/index.js # packages/core/src/modules/server/index.js # packages/gui/src/bridge/update/front.js # packages/gui/src/view/App.vue # packages/gui/vue.config.js # packages/mitmproxy/src/lib/proxy/middleware/overwall.js # packages/mitmproxy/src/lib/proxy/tls/FakeServersCenter.js
测试通过啦!!! |
好的,我去看下。 |
Ⅰ. 描述此PR的作用:
作为PR #391 的第二部分
TODO:
eslint
Ⅱ. 此PR修复了哪个issue吗?
fixes #390
Ⅲ. 界面变化截屏