No need to redirect to github.io, fix #35

gaospecial commented 3 years ago

原先是以 bookdown.org 为官方主页，而在 GitHub 发布的网页默认不在上面，因此需要强制重定向。

现在改用 GitHub Actions 之后，默认发布在 github.io ，因为 onload 事件前后的 location.host 是一样的，所以会在成功载入网页后不断刷新。

最严谨的方式可能是在 onload 事件所指定的函数上面做一个判断，如果 location.host 不是 github.io，则重定向到官方的发布页。

这里仅仅去掉了 onload 事件，单纯解决了反复重定向的问题。

PS：事实上只做了 4 个 commit，但是因为没有建立 branch，导致历次的 commit 在这里都可见。看来以后还是应该建立 branch 再提交。

gaospecial commented 3 years ago

代码要是不需要了可以直接删掉，不用注释掉。你们自己决定使用 github.io 版本还是 bookdown.org 版本。

图片的改动是怎么回事呢？如果不是必要的改动，建议不要混杂在同一个 PR 里。

请 @enixam 定夺吧。

在 #37 新建了 branch 后重新提交 PR，删掉了不需要的代码。

图片是因为本地的比较新，在 merge upstream/master 的时候加进来的。

难以理解的是，虽然使用了新的 branch 提交，但是结果仍然是全部的历史都列在了这里。我进行的是下面这些操作， @yihui 能否看看是我的工作流有问题吗？

$> git remote get-url upstream
# https://github.com/cosname/rmarkdown-guide.git

$> git fetch upstream
$> git commit -am "commit stated before merge"
$> git merge upstream/master
$> git commit -am "merge upstream/master" # after resloving conflicts

# work on a new branch
$> git branch disable-onload-refresh
$> git checkout disable-onload-refresh
$> git commit -am "disable onload refresh"  # after editing
$> git push --set-upstream origin disable-onload-refresh

# create a new PR #37

怎样才能创建一个干净的 PR 呢？

yihui commented 3 years ago

我没仔细看（现在太晚了，我要去睡觉了），不过我估计你的 master 分支已经包含了那些旧记录，你的新分支是建立在你的旧记录之上的。两个办法：要么删库重新 fork（本地库如果有新工作的话先别删）；要么在你的 master 分支里检出跟这边 master 分支最新的一次提交，然后强推到你的 master 分支（git push -f），顺了以后再在此基础上建新分支。

要是我没说清楚的话，请 @enixam 帮忙吧。

所以最后还是要牢记，凡是涉及到与他人合作的项目，最好都在新分支中进行新工作，主分支仅仅用来同步上游库。

cosname / rmarkdown-guide

No need to redirect to github.io, fix #35 #36