Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@BARMPlus
Copy link
Contributor

@BARMPlus BARMPlus commented Jul 6, 2023

No description provided.

@BARMPlus BARMPlus requested a review from LeoYuan as a code owner July 6, 2023 06:23
@LeoYuan
Copy link
Contributor

LeoYuan commented Jul 6, 2023

分开两个PR和commit会不会好点?

@BARMPlus
Copy link
Contributor Author

BARMPlus commented Jul 6, 2023

嗯,已分开

@LeoYuan
Copy link
Contributor

LeoYuan commented Jul 11, 2023

改了?我看还有 styleImport 插件的引入?

@LeoYuan
Copy link
Contributor

LeoYuan commented Jul 11, 2023

commit msg 也请同步改下~

@BARMPlus
Copy link
Contributor Author

创建的模版同步icejs模版,commit message 已经体现了
image

@LeoYuan
Copy link
Contributor

LeoYuan commented Jul 11, 2023

image

我意思是一个 commit 干一件事情,比如
commit1: chore: sync template icejs
commit2: chore: replace deprecated api

这样,commit timeline 比较清晰,对于需要 cherry pick 合入其他分支 / revert 回滚某个 commit 也比较方便

当然,PR 里可以罗列多个 commit,commit 是原子数据,PR 是业务/视图数据

@BARMPlus
Copy link
Contributor Author

done

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.

2 participants