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

Awesome Song! #392

Merged
merged 1 commit into from
Apr 18, 2015
Merged

Awesome Song! #392

merged 1 commit into from
Apr 18, 2015

Conversation

FangedTerror802
Copy link
Contributor

Add the best background music ever to the site! (It actually doesn't slow anything down too!)

@botwillacceptanything
Copy link
Owner

☑️ Voting procedure reminder:

To cast a vote, post a comment containing 👍:+1:, or 👎:-1:.
Remember, you must ⭐star this repo for your vote to count.

All comments within this discussion are searched for votes, regardless of the time of posting.
You can cast as many votes as you want, but only the last one will be counted.
(You may consider editing your comment instead of adding a new one.)
Comments containing both 👍up- and 👎down-votes are disregarded.
Comments containing monkeys are disregarded. 🐒 🐵 🙈 🙉 🙊
PR authors automatically count as a 👍 vote.

A decision will be made after this PR has been open for 15 minutes (plus/minus 10 percent, to avoid people timing their votes), and at least 8 votes have been made.
A supermajority of 65% is required for the vote to pass.

NOTE: the PR will be closed if any new commits are added after:
a738d6d

@botwillacceptanything
Copy link
Owner

👍

@ghost
Copy link

ghost commented Apr 18, 2015

👍

@coveralls
Copy link

Coverage Status

Coverage remained the same at 27.56% when pulling a738d6d on FangedTerror802:master into d682e7c on botwillacceptanything:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 27.56% when pulling a738d6d on FangedTerror802:master into d682e7c on botwillacceptanything:master.

@timweightman
Copy link
Contributor

👎
I don't believe that bots appreciate music as we humans do.
I also think that bot finds the rainbow / pink monstrosity that is its current web presence to be utterly confusing.
Bot most likely cares only of conveying its message and informing site viewers. We should simplify the UI/UX to be more bot-like.

@SpenserJ
Copy link
Contributor

👎 I agree with @timweightman. The interface is already polluted enough as it is, and I'd rather do away with the noise, and get back to just the bare statistics. Hell, I think a true interface for the bot should be JSON APIs, and everyone can build their own interfaces. Could be a neat feature to have the bot manage a separate repository.

@Prior99
Copy link
Contributor

Prior99 commented Apr 18, 2015

👍 moarrrr bullshit

@korczis
Copy link
Contributor

korczis commented Apr 18, 2015

👎

@dos1
Copy link
Contributor

dos1 commented Apr 18, 2015

I can see that this project is under a hostile takeover from people that don't understand it.

👍

@samis
Copy link
Contributor

samis commented Apr 18, 2015

👎
Make it not autoplay and I will upvote it. Autoplaying website music is hell.

@dos1
Copy link
Contributor

dos1 commented Apr 18, 2015

I agree, though not enough to downvote it 👍

@samis
Copy link
Contributor

samis commented Apr 18, 2015

Fair point. 👍

@botwillacceptanything
Copy link
Owner

👍 The vote passed! This PR will now be merged into master.


Tallies:
👍: 6 (66.6%)
👎: 3 (33.3%)

These users aren't stargazers, so their votes were not counted:

botwillacceptanything added a commit that referenced this pull request Apr 18, 2015
@botwillacceptanything botwillacceptanything merged commit fd1b43b into botwillacceptanything:master Apr 18, 2015
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.

None yet

10 participants