lita-pullrequests 0.0.1 → 0.0.2
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/README.md +1 -1
- data/lib/lita/handlers/pullrequests.rb +4 -4
- data/lita-pullrequests.gemspec +1 -1
- data/spec/lita/handlers/pullrequests_spec.rb +8 -6
- metadata +1 -1
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA1:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: 4dd5b417518a055224ceff19fc93b4855fa72a6e
|
|
4
|
+
data.tar.gz: 2e9da1a4c86869447103c22114875958ac284756
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: e2ea4f78b1e1f4df961474fea3e9ee0f20a6f7cba6c07b6641a4c49fce7fb796c54e8875a102aa4ddda34e11d5c7f0a32fa4c02094a2cb6783de97369bafdd7a
|
|
7
|
+
data.tar.gz: 322ad9afa953af2626987d06257429caf6b7b38799daf31c1c250a592634f77f7ed711f3b0d28fc6847cc160052714ceb1a02f722c6c7b0ad5cf82432baec5c0
|
data/README.md
CHANGED
|
@@ -18,7 +18,7 @@ Add the following configuration lines to your `lita_config`:
|
|
|
18
18
|
config.handlers.pullrequests.access_token = "a-github-api-access-token"
|
|
19
19
|
config.handlers.pullrequests.repo = "username/reponame"
|
|
20
20
|
config.handlers.pullrequests.review_label = "title of a label that represents a pr ready for review"
|
|
21
|
-
config.handlers.pullrequests.
|
|
21
|
+
config.handlers.pullrequests.merge_label = "title of a label that represents a pr ready for merge"
|
|
22
22
|
```
|
|
23
23
|
|
|
24
24
|
## Usage
|
|
@@ -1,24 +1,24 @@
|
|
|
1
1
|
module Lita
|
|
2
2
|
module Handlers
|
|
3
3
|
class Pullrequests < Handler
|
|
4
|
-
config :access_token, type: String, required: true
|
|
4
|
+
config :access_token, type: String, required: true
|
|
5
5
|
config :repo, type: String, required: true
|
|
6
6
|
config :review_label, type: String, required: false
|
|
7
7
|
config :merge_label, type: String, required: false
|
|
8
8
|
|
|
9
|
-
route(
|
|
9
|
+
route(/^(pull request( me)?)|(give me something to review)$/, :get_random_pr, command: true, help: {
|
|
10
10
|
"give me something to review" => "Shows you a random pull request that needs reviewing.",
|
|
11
11
|
"pull request (me)" => "Shows you a random pull request that needs reviewing."
|
|
12
12
|
})
|
|
13
13
|
|
|
14
|
-
route(
|
|
14
|
+
route(/^(summarize|all) pull requests$/, :list_all_pull_requests, command: true, help: {
|
|
15
15
|
"(summarize|all) pull requests" => "Lists all pull requests that need action."
|
|
16
16
|
})
|
|
17
17
|
|
|
18
18
|
|
|
19
19
|
# Helper method
|
|
20
20
|
def truncate(str, truncate_at, options = {})
|
|
21
|
-
return
|
|
21
|
+
return str unless str.length > truncate_at
|
|
22
22
|
|
|
23
23
|
omission = options[:omission] || '...'
|
|
24
24
|
length_with_room_for_omission = truncate_at - omission.length
|
data/lita-pullrequests.gemspec
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
Gem::Specification.new do |spec|
|
|
2
2
|
spec.name = "lita-pullrequests"
|
|
3
|
-
spec.version = "0.0.
|
|
3
|
+
spec.version = "0.0.2"
|
|
4
4
|
spec.authors = ["Taylor Lapeyre"]
|
|
5
5
|
spec.email = ["taylorlapeyre@gmail.com"]
|
|
6
6
|
spec.description = %q{A Lita handler to help you keep track of your pull requests.}
|
|
@@ -2,17 +2,19 @@ require "spec_helper"
|
|
|
2
2
|
|
|
3
3
|
describe Lita::Handlers::Pullrequests, lita_handler: true do
|
|
4
4
|
before do
|
|
5
|
-
Lita.config.handlers.pullrequests.access_token = "
|
|
5
|
+
Lita.config.handlers.pullrequests.access_token = ENV["GITHUB_ACCESS_TOKEN"]
|
|
6
6
|
Lita.config.handlers.pullrequests.repo = "taylorlapeyre/lita-pullrequests"
|
|
7
7
|
Lita.config.handlers.pullrequests.review_label = "Needs Review"
|
|
8
8
|
Lita.config.handlers.pullrequests.merge_label = "Ready To Merge"
|
|
9
9
|
end
|
|
10
10
|
|
|
11
|
-
it { is_expected.to route_command("pull request") }
|
|
12
|
-
it { is_expected.to route_command("pull request me") }
|
|
13
|
-
it { is_expected.to route_command("give me something to review") }
|
|
14
|
-
it { is_expected.to route_command("all pull requests") }
|
|
15
|
-
it { is_expected.to route_command("summarize pull requests") }
|
|
11
|
+
it { is_expected.to route_command("pull request").to(:get_random_pr) }
|
|
12
|
+
it { is_expected.to route_command("pull request me").to(:get_random_pr) }
|
|
13
|
+
it { is_expected.to route_command("give me something to review").to(:get_random_pr) }
|
|
14
|
+
it { is_expected.to route_command("all pull requests").to(:list_all_pull_requests) }
|
|
15
|
+
it { is_expected.to route_command("summarize pull requests").to(:list_all_pull_requests) }
|
|
16
|
+
|
|
17
|
+
it { is_expected.to_not route_command("all pull requests").to(:get_random_pr) }
|
|
16
18
|
|
|
17
19
|
it "can respond with a random pull request" do
|
|
18
20
|
send_command("give me something to review")
|