Skip to content

upgrade: 清理和升级依赖 #391

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.

Sign up for GitHub

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

Jump to bottom
Closed
starknt wants to merge 0 commits into from
Closed

upgrade: 清理和升级依赖 #391

starknt wants to merge 0 commits into from

Conversation

@starknt
Copy link
Contributor

@starknt starknt commented Nov 11, 2024 •
edited by wangliang181230
Loading

注：已拆分为 #395 和 #396 ，且两个PR均已合并

Ⅰ. 描述此PR的作用：

  • 清理无用的依赖项，并升级部分依赖项
  • 升级 nodejs
  • 测试

Ⅱ. 此PR修复了哪个issue吗？

fixes #390

@wangliang181230 wangliang181230 added this to the 1.9.0 milestone Nov 12, 2024
@wangliang181230 wangliang181230 changed the title 清理和升级依赖 upgrade: 清理和升级依赖 Nov 12, 2024
@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 12, 2024 •
edited
Loading

最好 nodejs 版本也升级，较新的LTS版本是 20.x 或 22.x 吗？
可以把 .github/目录下的两个脚本里的nodejs调上去进行测试。

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 12, 2024 •
edited
Loading

麻烦在 issue #390 里发个话，我把那个issue指派给你。^_^

@starknt starknt mentioned this pull request Nov 12, 2024
Closed
3 tasks
wangliang181230
wangliang181230 reviewed Nov 12, 2024
View reviewed changes
@starknt starknt marked this pull request as ready for review November 12, 2024 10:59
@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 12, 2024 •
edited
Loading

问题1（已解决）：

windows版的，启动不了代理服务，一启动就 exit 了：

点击查看截图

图片

@starknt
Copy link
Contributor Author

starknt commented Nov 12, 2024 •
edited
Loading

等我测试完再合并吧，改动太大了，有可能是幽灵依赖的问题

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 13, 2024

等我测试完再合并吧，改动太大了，有可能是幽灵依赖的问题

好的。

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 13, 2024 •
edited
Loading

问题2：

我在自己 fork 的仓库上，弄了个 run.yml，执行 npm run electron，测试直接运行DS的情况，Ubuntu环境报以下错误：

截图

图片

详细日志：

https://github.com/wangliang181230/dev-sidecar/actions/runs/11808871723/job/32898195861

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 13, 2024 •
edited
Loading

问题3：所有日志，保存在了 gui.log 里了，server.log 里只有一行日志，core.log 空的。

图片

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 13, 2024 •
edited
Loading

问题4：左上角的logo不见了

图片

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 13, 2024

问题1：

windows版的，启动不了代理服务，一启动就 exit 了

这个问题没有了，可以启动代理服务了。

@wangliang181230 wangliang181230 modified the milestones: 1.9.0, 2.0.0 Nov 13, 2024
@starknt
Copy link
Contributor Author

starknt commented Nov 13, 2024 •
edited
Loading

有几个包更新改动比较大，我先看看他们的更新情况吧，估计要花点时间

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 14, 2024

有几个包更新改动比较大，我先看看他们的更新情况吧，估计要花点时间

好的。

@wangliang181230 wangliang181230 marked this pull request as draft November 14, 2024 01:34
@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 14, 2024

对了，问一下，大量的代码格式调整，是什么工具弄的吗？

@starknt
Copy link
Contributor Author

starknt commented Nov 14, 2024

@antfu/eslint-config

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 14, 2024 •
edited
Loading

@antfu/eslint-config

可以配置 方法名 和 (...) 中间多一个空格吗？我觉得方法定义有个空格，在代码检索的时候，比较容易分别检索代码。
想检索方法调用代码时，搜索词可以不加空格，想找方法定义代码时，搜索词加空格。

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 14, 2024 •
edited
Loading

我先单独把大量的代码格式调整Commit一下，以突出这个PR里的变更代码吧。

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 14, 2024 •
edited
Loading

我把 github actions 脚本简化了，只使用一个 nodejs 版本，感觉没太必要弄2个版本。
另外，打印了一下python的版本号，供别人参考。

@starknt
Copy link
Contributor Author

starknt commented Nov 14, 2024

我打算后面重新开一个新的PR，这个PR的内容有点杂了，或者git push -f覆盖它

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 15, 2024

我打算后面重新开一个新的PR，这个PR的内容有点杂了，或者git push -f覆盖它

我觉得还好吧，我这边负责把格式化的代码先合并掉以后，这个PR的代码变更应该不多的。

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 15, 2024

你等我今天把格式化的代码先合了，再处理这个PR里遗留的问题，处理完就可以合了。

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 15, 2024

不过，也行吧，这个PR拆分掉，可能会更好一些。

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 15, 2024

能先提交一个 使用@antfu/eslint-config替换原来的eslint规则 的PR么。我复制了你的代码格式变更，现在master分支的最新代码，打包不了了。 (T.T)

@starknt
Copy link
Contributor Author

starknt commented Nov 15, 2024

你可以先把vue-cli的eslint的检查关掉

@wangliang181230
Copy link
Collaborator

wangliang181230 commented Nov 15, 2024 via email

@starknt
Copy link
Contributor Author

starknt commented Nov 15, 2024 •
edited
Loading

好，我先提交一个这个PR的前两个部分PR

@starknt starknt mentioned this pull request Nov 15, 2024
Merged
@wangliang181230 wangliang181230 added the v2.0.0 2.0.0版本 label Nov 15, 2024
@starknt starknt closed this Nov 15, 2024
@starknt starknt mentioned this pull request Nov 15, 2024
Merged
3 tasks
@wangliang181230 wangliang181230 added v2.0.0 2.0.0版本 and removed v2.0.0 2.0.0版本 labels Nov 18, 2024
@wangliang181230 wangliang181230 removed this from the 2.0.0 milestone Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wangliang181230 wangliang181230 wangliang181230 left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

【请求帮助】2.0.0 大版本更新，想将 NodeJS 和 各组件 的版本升级到较新版本，请求前端大佬帮助！！！

2 participants

@starknt @wangliang181230