Skip to content

作者接受重构型的pr吗 #23

@darkskygit

Description

@darkskygit

很高兴看到这个app开源,这是我很久以来一直想写但懒得去写的一类软件
此前我一直在使用相当老版本的一个rolan,昨天忽然发现只要它开着,我的核显就会有持续性的memory copy性能,让我的笔记本耗电增加了不少,实在不能忍了,于是花了半天翻找类似的软件,最后发现了该仓库

我编译并试用了软件,感觉使用体验上挺不错,不过我认为有一些可以改进的点,例如:

  1. rust包虽然使用了napi,但是采用手动require的方式使用,import进来是没有类型的
  2. 虽然使用了rust,但数据层仍然以过程式编程的方式写在js端,需要增加额外的依赖处理
  3. 大量的ipc调用类型都是手写的
  4. 缺少自动化测试

目前我在我的分支里已经做了初步的调整,主要是将仓库调整为monorepo,并将读取数据源逻辑先迁移到rust端,之后会把所有数据逻辑都弄到rust里:https://github.com/darkskygit/DawnLauncher

如果作者愿意接受重构型pr,我会将改动拆分为几个中型pr分批发过来

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions