Navigation Menu

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

Jump to bottom

Add repo settings #38

Merged
merged 2 commits into from May 7, 2019
Merged

Add repo settings #38

merged 2 commits into from May 7, 2019

Conversation

LinuxSuRen
Copy link
Member

No description provided.

@LinuxSuRen LinuxSuRen requested a review from a team May 6, 2019 03:28
@LinuxSuRen
Copy link
Member Author

See also https://github.com/probot/settings

@zacker330
Copy link
Contributor

这是加上 probot 了吗？

zacker330
zacker330 previously approved these changes May 6, 2019
View reviewed changes
Copy link
Contributor

@zacker330 zacker330 left a comment

Choose a reason for hiding this comment

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

LGTM 看样子已经安装好了 probot app 了。

@LinuxSuRen LinuxSuRen requested a review from a team May 7, 2019 01:07
@LinuxSuRen LinuxSuRen requested a review from zacker330 May 7, 2019 01:14
@donhui
Copy link
Member

donhui commented May 7, 2019

LGTM
期待它大显身手，哈哈

@donhui
Copy link
Member

donhui commented May 7, 2019

LGTM 看样子已经安装好了 probot app 了。

LGTM 貌似现在不起作用，不会 approve

@LinuxSuRen
Copy link
Member Author

这个不是指令机器人🤖️，只是把仓库的设置写的文件里了

@donhui
Copy link
Member

donhui commented May 7, 2019

Everything is code！

@LinuxSuRen
Copy link
Member Author

越来越多的内容和过程可以被 review 了

zacker330
zacker330 approved these changes May 7, 2019
View reviewed changes
Copy link
Contributor

@zacker330 zacker330 left a comment

Choose a reason for hiding this comment

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

LGTM

@LinuxSuRen LinuxSuRen merged commit 1fc7d6b into master May 7, 2019
@LinuxSuRen LinuxSuRen deleted the LinuxSuRen-patch-1 branch May 7, 2019 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zacker330 zacker330 zacker330 approved these changes

@donhui donhui donhui approved these changes

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
@LinuxSuRen @zacker330 @donhui