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

Remove bot's response to repos missing @gitcoinbot #1500

Merged
merged 1 commit into from
Jun 18, 2018

Conversation

arku
Copy link
Contributor

@arku arku commented Jun 18, 2018

Description

The gitcoinbot currently reacts with a heart emoji and a comment asking the maintainer to install gitcoinbot on the repo. The PR retains the heart emoji reaction and removes the comment.

Checklist
  • linter status: 100% pass
  • changes don't break existing behavior
  • commit message follows commit guidelines
Refers/Fixes

Fixes #1498

Copy link
Contributor

@mbeacom mbeacom left a comment

Choose a reason for hiding this comment

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

lgtm! Thanks! :shipit:

@mbeacom mbeacom merged commit 1f02864 into gitcoinco:master Jun 18, 2018
@codecov
Copy link

codecov bot commented Jun 18, 2018

Codecov Report

Merging #1500 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1500      +/-   ##
==========================================
+ Coverage    28.3%   28.31%   +<.01%     
==========================================
  Files         125      125              
  Lines        9235     9233       -2     
  Branches     1188     1188              
==========================================
  Hits         2614     2614              
+ Misses       6524     6522       -2     
  Partials       97       97
Impacted Files Coverage Δ
...itcoinbot/management/commands/get_notifications.py 0% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 687ecb9...bb76e36. Read the comment docs.

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.

Disable Gitcoin Bot Responses
2 participants