Skip to content
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

chore: use cz-git commitizen adapter #847

Merged
merged 3 commits into from
May 24, 2022
Merged

chore: use cz-git commitizen adapter #847

merged 3 commits into from
May 24, 2022

Conversation

Zhengqbbb
Copy link
Contributor

@Zhengqbbb Zhengqbbb commented May 23, 2022

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

💡 需求背景和解决方案

Hi 我是 cz-git commitizen 适配器的作者,偶尔之间我查看了这个项目,发现他非常适合使用cz-git, 来提升你们的日常开发

主要特点:

  1. 交互体验。支持搜索,和上下选择的交互模式,减少 commit 过程中出现的拼写错误
  2. 高度自定义,cz-git 提供了丰富的自定义功能,让 工具 更契合团队整体习惯
  3. 与 commitiint 联动,可以读取和编写配置在 commitiint 配置文件
  4. 轻量化:只有一个独立依赖,1.8 MB

欢迎尝试使用体验,有任何问题或需求可以告诉我

demo

随带一提

可以考虑删除 .husky/prepare-commit-msg 文件
我在提交的过程中发现了你们使用 强制转换的 git hook .husky/prepare-commit-msg
我个人认为这个是非常不好的。这会改变原有 git commit 命令的行为,并且在每次commit过后会自动打开编辑模式。这丧失了使用命令行工具提供的便利性
BTW: 使用cz-git 可以在最终提交确认时输入 e 进入编辑模式。直接回车就提交

📝 更新日志

  • 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

@PengYYYYY
Copy link
Collaborator

PengYYYYY commented May 23, 2022

ok, 欢迎贡献,等我空下来,我仔细看一下

@github-actions
Copy link
Contributor

github-actions bot commented May 23, 2022

完成

@PengYYYYY
Copy link
Collaborator

image

,DS_Store 这种可以忽略掉嘛

@PengYYYYY
Copy link
Collaborator

另外一点,如果可以拿git缓冲区里面修改了的文件路径,自动加入到scope里面会更好

@Zhengqbbb
Copy link
Contributor Author

Zhengqbbb commented May 23, 2022

@PengYYYYY

DS_Store 这种可以忽略掉嘛

这可能可以自动添加🤔。不过我添加这一排除在配置中,可以自行添加修改。

另外一点,如果可以拿git缓冲区里面修改了的文件路径,自动加入到scope里面会更好

很棒的提议!这可以自定义实现,我大致完成了整体效果你可以看一下,可以根据团队习惯需要进行调整
defaultScope,这一配置项我那时候的想法和设计并不是自动添加,而是通过置顶的方式。

demo

@PengYYYYY
Copy link
Collaborator

ok

@PengYYYYY PengYYYYY merged commit 9e15b79 into Tencent:develop May 24, 2022
@Zhengqbbb
Copy link
Contributor Author

Zhengqbbb commented May 24, 2022

ok

@PengYYYYY

const scopeComplete = execSync("git status -s | grep 'src' 2> /dev/null")
.toString()
.trim()
.split('\n')
.find((r) => r.indexOf('M ') !== -1)
?.match(/src\/(\S*)\//)?.[1];

这个地方还是要改一下比较好。命令处理 | 兼容windows enjoy it~~

// precomputed scope
const scopeComplete = execSync('git status -s || true')
  .toString()
  .trim()
  .split('\n')
  .find((r) => r.indexOf('M  ') !== -1)
  ?.replace(/(\/|\\\\)/g, '%%')
  ?.match(/src%%((\w|-)*)/);

image

@k1nz
Copy link
Contributor

k1nz commented May 24, 2022

image
提交不了了呢

@Zhengqbbb
Copy link
Contributor Author

image 提交不了了呢

https://github.com/Tencent/tdesign-vue-next/pull/867/files

可以看一下这个PR,试一下更改一下配置

手里目前没有windows电脑可以 进行测试,之前提交忘记做兼容了

@k1nz
Copy link
Contributor

k1nz commented May 24, 2022

image 提交不了了呢

https://github.com/Tencent/tdesign-vue-next/pull/867/files

可以看一下这个PR,试一下更改一下配置

手里目前没有windows电脑可以 进行测试,之前提交忘记做兼容了

刚看到,加上去可以了

@Zhengqbbb
Copy link
Contributor Author

image 提交不了了呢

https://github.com/Tencent/tdesign-vue-next/pull/867/files
可以看一下这个PR,试一下更改一下配置

手里目前没有windows电脑可以 进行测试,之前提交忘记做兼容了

刚看到,加上去可以了

有权限的话,可以帮忙过一下PR~ 🤩🤩🤩

@github-actions github-actions bot mentioned this pull request May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants