Sutou Kouhei (45e56374) at 08 Apr 05:15

Sutou Kouhei (b263e939) at 08 Apr 00:12

Sutou Kouhei (0aee8c5b) at 08 Apr 00:10

Sutou Kouhei (0aee8c5b) at 08 Apr 00:09

Sutou Kouhei (f0e03483) at 29 Aug 05:18

Sutou Kouhei (809c4b4d) at 29 Aug 05:17

Sutou Kouhei (ad0a215a) at 22 Jul 02:15

Sutou Kouhei (4dac5830) at 22 Jul 02:14

Sutou Kouhei (e2cd68bf) at 22 Jul 01:54

Sutou Kouhei (eadde3ad) at 22 Jul 00:57

Sutou Kouhei (36a217cd) at 22 Jul 00:55

Sutou Kouhei (04fb513b) at 22 Jul 00:55

Sutou Kouhei (8bc73a05) at 22 Jul 00:55

Sutou Kouhei (34fdbbd9) at 22 Jul 00:45

Thanks for your review and merging!

Sutou Kouhei (0aee8c5b) at 04 Apr 08:13

複数のwebhook-end-pointを設定可能にしました。

設定ファイルの互換性がなくなります。新しいフォーマットは次のようになります。

"https://example.com/webhook1":
  events:
    - PushEvent
    - PullRequestEvent
  repositories:
    - apache/arrow
    - apache/arrow-julia
"https://example.com/webhook2":
  events:
    - PushEvent
    - PullRequestEvent
  repositories:
    - fluent/fluentd

また動作未確認ですが、従来はEventの種類ごとにRepositoryを登録する設定が期待通り動かないように見えます。 従来の設定で次のようなものを例として考えると、

events:
  PushEvent:
    repositories:
      - repo-A
    webhook-end-point: end-point-A
  PullRequestEvent:
    repositories:
      - repo-B
    webhook-end-point: end-point-B

end-point-Aへの送信にrepo-Bのイベントも含まれる可能性があるように思います。

今回の修正でこの問題も解決するはずです。

Fixed at 96cec57c

Thanks!

I see!

That makes the code more readable and manageable!

I will fix it.

How about using instance variables instead of Hash?

        @xmlua_events = Events.new([], [])
        http_server.mount("/repos/clear-code/xmlua/events", Handler, @xmlua_events)