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

Add 更详细的 GitHub 与 git 使用指南 #4017

Closed
Cubik65536 opened this issue Jun 25, 2022 · 5 comments
Closed

Add 更详细的 GitHub 与 git 使用指南 #4017

Cubik65536 opened this issue Jun 25, 2022 · 5 comments
Labels
Content Request / 内容请求 New feature or request

Comments

@Cubik65536
Copy link
Contributor

页面英文名

No response

我希望能添加的内容是

近期我发现有多个 Pull Request 没有在收到更改建议后,通过向请求拉取的分支 commit 来改正问题,而是直接开启了新分支进行更改。这样会导致一个 Pull Request 应该更改的内容被分布在了多个 Pull Request 中。所以我希望可以增加更详细的 GitHub/git 使用指南,这或许会让第一次参加开源项目的贡献者们更难上手,但这无疑可以优化整个社区的工作流程。

PS. 目前大部分类似的情况似乎都是因为贡献者参考了 如何参与 - 在 GitHub 上编辑 章节。这个章节的操作在仅进行一次更改的情况下并不会有任何问题,但是在需要多次 commit(例如根据 review 进行改进)的时候,由于 GitHub 并不会向 fork 已有的分支 commit,而是开一个新的分支,但实际上贡献者们需要直接更改自己的 fork 的相关分支,因此造成了问题。

我了解到的相关参考资料有

No response

@Cubik65536 Cubik65536 added the Content Request / 内容请求 New feature or request label Jun 25, 2022
@welcome
Copy link

welcome bot commented Jun 25, 2022

感谢你对 OI Wiki 的关注!记得在 Issue 中表达清楚自己的意思哦~

@CoelacanthusHex
Copy link
Member

CoelacanthusHex commented Jul 13, 2022

事实上这些问题在 GitHub 的文档是有提到的,比如更新 PR 的问题:

在创建拉取请求后,您可以从主题分支推送提交,以将它们添加到现有的拉取请求。 这些提交将以时间顺序显示在您的拉取请求中,在 "Files changed"(更改的文件)选项卡中可以看到更改。
https://docs.github.com/cn/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests

所以可能只添加一个链接就够了

GitHub Docs
拉取请求可让您在 GitHub 上向他人告知您已经推送到仓库中分支的更改。 在拉取请求打开后,您可以与协作者讨论并审查潜在更改,在更改合并到基本分支之前添加跟进提交。

@Cubik65536
Copy link
Contributor Author

Cubik65536 commented Jul 13, 2022

事实上这些问题在 GitHub 的文档是有提到的,比如更新 PR 的问题:

在创建拉取请求后,您可以从主题分支推送提交,以将它们添加到现有的拉取请求。 这些提交将以时间顺序显示在您的拉取请求中,在 "Files changed"(更改的文件)选项卡中可以看到更改。
https://docs.github.com/cn/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests

所以可能只添加一个链接就够了

GitHub Docs**关于拉取请求 - GitHub Docs**拉取请求可让您在 GitHub 上向他人告知您已经推送到仓库中分支的更改。 在拉取请求打开后,您可以与协作者讨论并审查潜在更改,在更改合并到基本分支之前添加跟进提交。

感觉可以,不过应该还是得稍微多加一些解释,因为新手应该很难马上理解什么是 主题分支,我目前正在准备写相关内容。

GitHub Docs
拉取请求可让您在 GitHub 上向他人告知您已经推送到仓库中分支的更改。 在拉取请求打开后,您可以与协作者讨论并审查潜在更改,在更改合并到基本分支之前添加跟进提交。

@Cubik65536
Copy link
Contributor Author

提醒一下,我正在编写相关内容

@Cubik65536
Copy link
Contributor Author

该内容已于 PR #4067 增加,且已被合并

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content Request / 内容请求 New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants