finish translating contributing.md #94
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mudongliang
reviewed
Jul 12, 2024
sources/syzkaller/contributing.md
Outdated
|
|
||
| In case this is your first pull request to syzkaller, you will need to [sign Google CLA](https://cla.developers.google.com/) | ||
| and add yourself to [AUTHORS](/AUTHORS)/[CONTRIBUTORS](/CONTRIBUTORS) files in the first commit. | ||
| 如果这是你第一次向 syzkaller 提交拉取请求，你需要[签署 Google CLA](https://cla.developers.google.com/)。 |
mudongliang
reviewed
Jul 12, 2024
sources/syzkaller/contributing.md
Outdated
|
|
||
| Provide enough documentation for other users to use the new feature. | ||
| 为其他用户使用新功能提供足够的文档。 |
There was a problem hiding this comment.
改成这个 “为了其他用户更方便使用新特性，请提供足够的文档说明。”，如何？
mudongliang
reviewed
Jul 12, 2024
sources/syzkaller/contributing.md
Outdated
| ``` | ||
|
|
||
| `dir/path` is a relative path to the main dir this commit changes | ||
| (look at examples in the [commit history](https://github.com/google/syzkaller/commits/master)). | ||
| If several packages/dirs are significantly affected, then the following format is allowed: | ||
| If several packages/dirs are significantly affected, then the following format is allowed:是本次提交更改的主目录的相对路径。（参见 [commit history](https://github.com/google/syzkaller/commits/master) 中的示例）。如果多个软件包/目录受到重大影响，则允许使用以下格式： |
mudongliang
reviewed
Jul 12, 2024
mudongliang
reviewed
Jul 12, 2024
sources/syzkaller/contributing.md
Outdated
| If you don't yet have Docker installed, see [documentation](https://docs.docker.com/engine/install), | ||
| in particular regarding enabling [sudo-less](https://docs.docker.com/engine/install/linux-postinstall) | ||
| Docker (Googlers see go/docker). | ||
| 开发 syzkaller 需要安装大量工具（Go 工具链、C/C++ 交叉编译器、golangci-lint 等）。安装所有这些工具可能会很麻烦，例如由于软件包损坏/缺失。[syz-env](/tools/syz-env) 提供了一个基于 Docker 容器的密封开发环境。如果尚未安装 Docker，请参阅[文档](https://docs.docker.com/engine/install)、特别是关于启用 [sudo-less](https://docs.docker.com/engine/install/linux-postinstall) 的 Docker（Googlers 参见 go/docker）。 |
mudongliang
reviewed
Jul 12, 2024
sources/syzkaller/contributing.md
Outdated
| - 将更改提交到本地。为此，你需要对所有更改的文件运行 `git add`，例如 `git add sys/linux/sys.txt`。你可以运行 `git status` 查看有哪些文件被修改/创建。当所有文件都添加完毕后（`git status` 显示 `Changes not staged for commit` 部分没有文件，并且 `Untracked files` 部分没有相关文件），运行 `git commit` 并在你的编辑器中输入提交描述。 | ||
| - 在本地运行测试（`make install_prerequisites`，然后执行 `make presubmit`）。 | ||
| - 使用 `git push my-origin my-branch` 将提交推送到 github 上的分叉。 | ||
| - 导航至 [github.com/google/syzkaller](https://github.com/google/syzkaller)，你会看到绿色的 `Compare & pull request` 按钮，按下它。然后按 `创建拉取请求`。现在你的拉取请求应该会出现在[拉取请求页面](https://github.com/google/syzkaller/pulls)上。 |
There was a problem hiding this comment.
此处Github提示最好是可以统一，要么都是英文，要么都是中文。按照现在看，“创建拉取请求”是中文，其他提示是英文
|
@QGrain 请按照review建议进行修改 |
好的 👌 |
QGrain
force-pushed
the
contributing-translated
branch
from
a0cd2b1
to
61d180c
Compare
QGrain
force-pushed
the
contributing-translated
branch
from
61d180c
to
7d1b970
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.