-
Notifications
You must be signed in to change notification settings - Fork 6
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
support @ user after regression #63
Conversation
Thanks @davidx! Settings---
minimum_reviewers: 1
build_steps:
- bundle install
- bundle exec ruby test/test.rb
merge: true
org_mode: true
timeout: 1805
shell: "/bin/bash"
env:
BUNDLE_GEMFILE: Gemfile |
Looks good! 👍✅ MERGE
✅ BUNDLE_INSTALL
✅ BUNDLE_EXEC_RUBY_TEST/TEST.RB
⬜ 0 of 1 Code reviews from organization basho-labs |
+1 |
✅ 1 of 1 Code reviews from organization basho-labs
|
Merging and closing this pr |
Successfully merged basho-labs/thumbs/pulls/63 (6fb1713 on to master) ---
:sha: f1f0c2263e547f009ab4a19e94cf6caaac55ec48
:merged: true
:message: Pull Request successfully merged
|
This re-introduces an optional '@' preceding the thumbot commands. Currently matching /^thumbot and thus ignoring @thumbot mentions.
Example of @thumbot mention fail: basho/riak_kv#1560 (comment)