Skip to content

add details running on Windows#690

Merged
zhongjiajie merged 5 commits into
apache:masterfrom
yimaixinchen:windows-note
Feb 16, 2022
Merged

add details running on Windows#690
zhongjiajie merged 5 commits into
apache:masterfrom
yimaixinchen:windows-note

Conversation

@yimaixinchen
Copy link
Copy Markdown
Contributor

No description provided.

@yimaixinchen
Copy link
Copy Markdown
Contributor Author

this is not running on cmd.exe as ADMINISTRATOR.
d4bf2036b91a42559c5f6918ff6dfbef-60872

this is running on cmd.exe as ADMINISTRATOR.
捕获

Comment thread README.md Outdated
Comment thread README.md Outdated
yimaixinchen and others added 2 commits February 14, 2022 22:18
Co-authored-by: QuakeWang <45645138+QuakeWang@users.noreply.github.com>
Co-authored-by: QuakeWang <45645138+QuakeWang@users.noreply.github.com>
@yimaixinchen
Copy link
Copy Markdown
Contributor Author

Thanks for your suggestion.I fix it.

Comment thread README.md Outdated
@zhongjiajie zhongjiajie added develop Develop section change enhancement New feature or request labels Feb 15, 2022
@zhongjiajie zhongjiajie self-requested a review February 15, 2022 03:17
@zhongjiajie
Copy link
Copy Markdown
Member

Thank @yimaixinchen for the PR, and @QuakeWang for reviewing

Co-authored-by: QuakeWang <45645138+QuakeWang@users.noreply.github.com>
QuakeWang
QuakeWang previously approved these changes Feb 16, 2022
Copy link
Copy Markdown
Contributor

@QuakeWang QuakeWang left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread README.md Outdated
Copy link
Copy Markdown
Member

@zhongjiajie zhongjiajie left a comment

Choose a reason for hiding this comment

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

Looking good and thanks @yimaixinchen

@zhongjiajie zhongjiajie merged commit 13eab35 into apache:master Feb 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

develop Develop section change enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants