search

ocaml-opam / Camelus

Bot posting reports on opam-repository pull-request using a web hook
Other
18 stars 9 forks source link

Stay quieter #9

Open dsheets opened 8 years ago

dsheets commented 8 years ago

From ocaml/opam-repository#5995:

I find these notifications quite useful, since Travis doesnt send a message when it successfully finishes. There's absolutely no harm in a note from a package submitter saying that they believe that a PR is ready to merge since it passes CI, especially since it sometimes takes multiple pushes to get it right.

I find this very noisy, and I'm considering removing myself from watching this repository. If we follow this pattern we get at minimum of 4 notification per package addition:

  1. The initial PR
  2. Camelus
  3. The PR person says it's all green
  4. The merge notification

wouldn't it be possible to change this so that Camelus reports directly only if it fails and otherwise once travis succeded or failed ?

dsheets commented 8 years ago

This event configuration is possible with ocaml-github. I don't believe any additional bindings are required.

dbuenzli commented 8 years ago

Cool, thanks David.