Skip to content

Linkify "ChaoBot's logs"#403

Merged
chaosbot merged 2 commits intoChaosthebot:masterfrom
syvb:master
May 30, 2017
Merged

Linkify "ChaoBot's logs"#403
chaosbot merged 2 commits intoChaosthebot:masterfrom
syvb:master

Conversation

@syvb
Copy link
Contributor

@syvb syvb commented May 30, 2017

Makes the text "chaosbot's logs" in the README a link to the logs, where people can find the meritocracy information.

@rudehn
Copy link
Contributor

rudehn commented May 30, 2017

I would add a search term so people know how to find it in the logs.

@syvb
Copy link
Contributor Author

syvb commented May 30, 2017

@PlasmaPower you also have to react/comment 👍 in order @chaosbot to consider your vote -- the meritocracy commit made it so reviews only satisfy the meritocracy, not act as a vote.

@PlasmaPower
Copy link
Contributor

@Smittyvb Yeah I forgot to vote. Both are done now.

Copy link
Member

@phil-r phil-r left a comment

Choose a reason for hiding this comment

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

Test

Copy link
Member

@phil-r phil-r left a comment

Choose a reason for hiding this comment

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

🗑 Can't delete this ;(

@PlasmaPower
Copy link
Contributor

@phil-r did it work?

@phil-r
Copy link
Member

phil-r commented May 30, 2017

@PlasmaPower I was just checking API resonses to review #396

@PlasmaPower
Copy link
Contributor

@phil-r are they ordered correctly?

@phil-r
Copy link
Member

phil-r commented May 30, 2017

@PlasmaPower last one at the bottom

@PlasmaPower
Copy link
Contributor

Great!

@chaosbot
Copy link
Collaborator

🙆‍♀️ PR passed with a vote of 16 for and 0 against, a weighted total of 15.5 and a threshold of 6.5, and a current meritocracy review.

See merge-commit ddfdb84 for more details.

@chaosbot chaosbot merged commit ddfdb84 into Chaosthebot:master May 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants