野迂迂

Results 33 comments of 野迂迂

> > > @dtinsight/molecule 已经正常安装了, 并且yarn start可以正常运行。 是不是jest和cra生成的项目使用esm模块有不兼容啊? 是滴,因为 jest 是跑在 Node.js 的环境里的,目前 Node.js 对 esm 的支持还是实验性的功能。 如果有想要用 Molecule 测试 molecule 的话,目前理论上应该有两种办法, 1. 一种是黑盒思路,因为 Molecule 项目本身已经有足够的测试覆盖率,所以确保你自身项目的代码测试通过即可,Molecule 用到的 API 部分都通过 mock...

目前来说暂时没有最佳实践,针对 extensions 中的部分实现可以抽一个纯函数出来进行测试,但是大部分控件的配置方法暂时没什么特别好的办法,后面我们考虑这一块的内容

> 如果不能比较方便的单元测试,对于刚上手的新人,是不是需要频繁在浏览器里面看console试错来熟悉api。 其实不用,如果是 ts 的项目,那在 IDE 里面会有提示的,我们所有的函数都有补充文字说明,参数的命名还是比较符合规范的。 除此之外,还可以查 [API 文档](https://dtstack.github.io/molecule/docs/next/api/),所有 [API 的风格](https://dtstack.github.io/molecule/docs/next/contributing#naming)也是保持统一,确保用户的上手

We need a release script to do the following things: 1. 检查 git 状态(确保暂存区没有更改) 2. 检查分支状态(确保 release 的命令仅在 main 分支执行) 3. 检查 remote 状态(确保当前仓库是最新的) 4. 检查 npm publish 的地方(确保是 `https://registry.npmjs.org/`)...

评估了一下有破坏性变更,放到下个 minor 版本做

技术选型:pnpm + turbo 拆依赖包如下: - `@dtinsight/molecule` - 用户主要引入的包 - `@dtinsight/molecule/glue` - 抽象出 molecule 的开发范式,主要为 ide 服务 - `@dtinsight/molecule/ide` - Molecule 用到的大部分业务组件以及 services controllers - `@dtinsight/molecule/ui` - Molecule 抽象出来的组件 - `@dtinsight/molecule/website`...

> @mortalYoung `webiste` 应该不用做 package 的抽象。拆包更多还是为了后面需要独立发包来考虑。 `extensions` 我觉得可以多考虑下。 那我再改下

看了一下还是挺大的,🤣 建议是 review 的时候 select commit 一下,每次 commit 都只做一件事,应该会直观一点🥶

[theme.ts#L73](https://github.com/umijs/dumi/blob/master/packages/preset-dumi/src/plugins/features/theme.ts#L73) 这里的逻辑和目前自定义编译组件的逻辑无法自洽,需要做改造,看下如何改比较好 cc @PeachScript

> ⭐️⭐️ rehypeIsolation基础样式隔离,参考 [dumi v1](https://github.com/umijs/dumi/blob/d75c7dfa04b80268ec684ea1197212e1f3a67bfb/packages/preset-dumi/src/transformer/remark/embed.ts) 重写 🙋我认领一个这个 不过这个参考是不是写错了 应该是这个吧 [dumi isolation.ts](https://github.com/umijs/dumi/blob/d75c7dfa04b80268ec684ea1197212e1f3a67bfb/packages/preset-dumi/src/transformer/remark/isolation.ts)