add support for Baidu Music & add features for Xiami #136

Closed
wants to merge 12 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

twlz0ne commented Mar 26, 2013

PS: 在 pull request 的时候能把不相关的 commit去掉吗?
PPS: 如何在标题和消息正文中链接 fork 仓库的 issues

Owner

soimort commented Mar 26, 2013

这个我可以手动merge。

PS> 一般来说提交pull request可以这么做:新建一个branch,从上游库git rebase(不用git merge,因为这会增加一个额外的commit),然后做相应的修改,最后从这个branch来提交pull request。pull request被merge或者关闭之后,这个branch可以直接删除。
如果直接从head branch的master提交pull request,而master又有base branch的历史中不包含的commits时,pull request就会自动把这些commits全部包括进来(像你上面这样)。

PPS> twlz0ne/you-get#1

soimort closed this Mar 26, 2013

Contributor

twlz0ne commented Mar 28, 2013

我本地有两个分支:

 master  # 用于保存自己的一些features
 sync    # 用于与上游仓库同步

目前 sync 比 upstream/master 少一次提交,尝试:

git fetch upstream
git checkout sync
git rebase upstream/master

之后,产生了大量冲突,而且变成了 detached HEAD。

Contributor

twlz0ne commented Mar 28, 2013

查看了一下 sync 和 upstream/master 的提交记录,最近3次sha1都不一样。
解决冲突,接着git rebase --continue,然后git rebase --skip,如此重复几次之后,终于正常了。

Owner

soimort commented Mar 28, 2013

因为同样的修改commit时间不同,SHA1也不同……
直接git reset --hard应该更省事

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment