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.

Sign up for GitHub

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

Jump to bottom

add git cmd #63

Merged
merged 8 commits into from
Oct 14, 2020
Merged

add git cmd #63

merged 8 commits into from
Oct 14, 2020

Conversation

lxsz
Copy link
Contributor

@lxsz lxsz commented Aug 29, 2020

No description provided.

skywind3000
skywind3000 reviewed Sep 10, 2020
View reviewed changes
Copy link
Owner

@skywind3000 skywind3000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请看一下 review

tools/git.txt Outdated Show resolved Hide resolved
@skywind3000
Copy link
Owner

多谢更新，麻烦对齐一下注释，你有的地方用的 tab 有的地方用的空格，命令和注释中间请用空格，这样对齐看起来才一致，现在你提交的几行内容，在 github 上面看起来注释左左右右的，希望整理一下。

lxsz
lxsz commented Sep 21, 2020
View reviewed changes
Copy link
Contributor Author

@lxsz lxsz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

我好像不知道怎么提交了，看下原来空格也没对齐呀

skywind3000
skywind3000 reviewed Sep 21, 2020
View reviewed changes
Copy link
Owner

@skywind3000 skywind3000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@skywind3000 skywind3000 merged commit 380ccb6 into skywind3000:master Oct 14, 2020
@skywind3000
Copy link
Owner

OK merge 了。

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

@skywind3000 skywind3000 skywind3000 left review comments

@anxy0625 anxy0625 anxy0625 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@lxsz @skywind3000 @anxy0625