Skip to content

Bug Fix#4

Open
RohanBh wants to merge 4 commits into
mdgspace:masterfrom
RohanBh:bug/c-plus-plus
Open

Bug Fix#4
RohanBh wants to merge 4 commits into
mdgspace:masterfrom
RohanBh:bug/c-plus-plus

Conversation

@RohanBh

@RohanBh RohanBh commented Jun 3, 2017

Copy link
Copy Markdown
Member

This commit if applied will fix a bug which increases the score of c if a string "c++" is encountered in messages.

@csoni111

csoni111 commented Jun 5, 2017

Copy link
Copy Markdown
Member

On typing c++, bot shall give a response Sorry, You can't give ++ or -- to yourself.. This will be weird, don't you think?

@csoni111

csoni111 commented Jun 5, 2017

Copy link
Copy Markdown
Member

A better option will be if we only allow the ++ -- on usernames available. Check if you can figure out a way to do that @RohanBh !

@RohanBh

RohanBh commented Jun 6, 2017

Copy link
Copy Markdown
Member Author

ok I will work on it

@RohanBh RohanBh force-pushed the bug/c-plus-plus branch 2 times, most recently from 18f3375 to ce5d81b Compare June 10, 2017 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants