Skip to content
This repository has been archived by the owner on Aug 5, 2024. It is now read-only.

Stay quieter #9

Open
dsheets opened this issue Mar 15, 2016 · 2 comments
Open

Stay quieter #9

dsheets opened this issue Mar 15, 2016 · 2 comments

Comments

@dsheets
Copy link

dsheets commented Mar 15, 2016

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
Copy link
Author

dsheets commented Mar 15, 2016

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

@dbuenzli
Copy link

Cool, thanks David.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants