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

Adding more replies to 8ball and rate answers #435

Merged
merged 1 commit into from
Oct 24, 2019

Conversation

guipleite
Copy link
Contributor

Adding more replies to 8ball and rate answers for issue #117

Copy link
Contributor

@13xforever 13xforever left a comment

Choose a reason for hiding this comment

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

Some answers need more work or should be removed

CompatBot/Commands/Misc.cs Outdated Show resolved Hide resolved
CompatBot/Commands/Misc.cs Outdated Show resolved Hide resolved
CompatBot/Commands/Misc.cs Outdated Show resolved Hide resolved
CompatBot/Commands/Misc.cs Outdated Show resolved Hide resolved
CompatBot/Commands/Misc.cs Outdated Show resolved Hide resolved
CompatBot/Commands/Misc.cs Outdated Show resolved Hide resolved
@guipleite
Copy link
Contributor Author

I Implemented the requested changes in my last commit

@13xforever
Copy link
Contributor

Not sure you pushed the right changes, most answers just have a bunch of underscores now

@guipleite
Copy link
Contributor Author

guipleite commented Oct 24, 2019

Crap, sorry about that
I fixed it now

CompatBot/Commands/Misc.cs Outdated Show resolved Hide resolved
CompatBot/Commands/Misc.cs Outdated Show resolved Hide resolved
CompatBot/Commands/Misc.cs Outdated Show resolved Hide resolved
@13xforever 13xforever added the fun stuff things to entertain the crowds label Oct 24, 2019
@13xforever 13xforever added this to the v2.14 milestone Oct 24, 2019
@13xforever
Copy link
Contributor

13xforever commented Oct 24, 2019

can you squash all of this into one single commit? if not it's ok, I'd simply prefer to have this as an atomic change in the history

@13xforever
Copy link
Contributor

I think you pushed unmerged version with diffs for conflicted lines

@13xforever
Copy link
Contributor

thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fun stuff things to entertain the crowds
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants