Skip to content

use logbot to log our IRC channel #190

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

Merged
merged 1 commit into from
Aug 23, 2018
Merged

use logbot to log our IRC channel #190

merged 1 commit into from
Aug 23, 2018

Conversation

japaric
Copy link
Member

@japaric japaric commented Aug 22, 2018

This a proposal to log our IRC channel using logbot.

This proposal (PR) needs at least 10 votes (approvals) from @rust-embedded/all
to be accepted. After the proposal has been accepted we'll proceed to implement
it.

@rust-embedded/all please vote on this proposal using pull request reviews.
Or if you have a concern leave a comment.

closes #171

@japaric japaric requested review from dylanmckay, jcsoo and a team as code owners August 22, 2018 11:35
@japaric japaric added the needs-decision This RFC or PR needs to be approved by the majority of reviewers before it's merged label Aug 22, 2018
@jamesmunns jamesmunns self-requested a review August 22, 2018 20:46
@v-thakkar v-thakkar self-requested a review August 22, 2018 21:20
@japaric
Copy link
Member Author

japaric commented Aug 23, 2018

10 people have approved this so this proposal has been approved.

Logging is effective as of now: https://mozilla.logbot.info/rust-embedded

bors r+

@bors
Copy link
Contributor

bors bot commented Aug 23, 2018

👎 Rejected by label

@japaric japaric removed the needs-decision This RFC or PR needs to be approved by the majority of reviewers before it's merged label Aug 23, 2018
@japaric
Copy link
Member Author

japaric commented Aug 23, 2018

bors r+

bors bot added a commit that referenced this pull request Aug 23, 2018
190: use logbot to log our IRC channel r=japaric a=japaric

This a proposal to log our IRC channel using logbot.

This proposal (PR) needs at least 10 votes (approvals) from @rust-embedded/all
to be accepted. After the proposal has been accepted we'll proceed to implement
it.

@rust-embedded/all please vote on this proposal using [pull request reviews].
Or if you have a concern leave a comment.

[pull request reviews]: https://help.github.com/articles/about-pull-request-reviews/

closes #171

Co-authored-by: Jorge Aparicio <jorge@japaric.io>
@bors
Copy link
Contributor

bors bot commented Aug 23, 2018

Build succeeded

@bors bors bot merged commit 5afff94 into master Aug 23, 2018
@bors bors bot deleted the logbot branch August 23, 2018 11:18
@japaric japaric added the decision-accepted We voted on this proposal and accepted it label Aug 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
decision-accepted We voted on this proposal and accepted it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IRC logger hosted or maintained by us