Closed conglinyizhi closed 1 year ago
好的,我会尽快应用这些修改
完成,按钮已经更换为一个基于 Swicth
组建封装的 SwicthItem
并且更换了放置位置。
我不认为新增一个开关是新的功能,因此,我没有增加次版本号
PR的标题用于描述功能,比如 feat: add copy switch button 或 fix: support pnpm 最终会变成一个 git commit,这个 版本号不用你自己操作的,由 GitHub Action 自动生成。 另外麻烦把一些没必要的代码注释去掉哇, // thanks
我已经修改了 PR 的标题,但是我不清楚您描述的“没必要的代码”指的是哪些,是类似图片中的这种可以让编辑器展示更多提示的注释吗?
hooks/notes/lark.ts
中的 but why
注释?这个注释的原因是我目前不明白传入的数据中会多出来一个换行符,这个开头的换行符会导致问题 #54 ,所以我在某次修改的时候加入了这一行注释,考虑到这行注释可能会对其他贡献者产生麻烦,我也可以删除。期待您的回复。
好的,我会尽快修改,很抱歉我有一个差劲的编码习惯
对于 markdown 我在用过几次功能后的确有类似的想法,在删除那些注释后我会加上的
pnpm install
(fixed by installing several packages)Switch
component instead) and test it locally.gitignore
file to tell git to ignore pnpm's locked filesIn the future, can I fill in information in Chinese? Just when submitting a pull request. As Chinese can better express my intentions, English may lead to semantic bias, I look forward to your reply.
pnpm
安装的问题(解决方式是新增加了几个软件包).gitignore
文件中忽略pnpm
的 lock 文件